Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

rgbds: 0.5.1 -> 0.5.2 #149232

Merged
merged 3 commits into from
Jan 3, 2022
Merged

rgbds: 0.5.1 -> 0.5.2 #149232

merged 3 commits into from
Jan 3, 2022

Conversation

r-ryantm
Copy link
Contributor

@r-ryantm r-ryantm commented Dec 7, 2021

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/rgbds/versions.

meta.description for rgbds is: "A free assembler/linker package for the Game Boy and Game Boy Color"

meta.homepage for rgbds is: "https://rgbds.gbdev.io/"

meta.changelog for rgbds is: ""

Updates performed
  • Version update
To inspect upstream changes
Impact
Checks done (click to expand)
  • built on NixOS
  • The tests defined in passthru.tests, if any, passed
  • /nix/store/bfq4av32wahnp792adx4zba5aznv6ja7-rgbds-0.5.2/bin/rgblink passed the binary check.
  • /nix/store/bfq4av32wahnp792adx4zba5aznv6ja7-rgbds-0.5.2/bin/rgbasm passed the binary check.
  • /nix/store/bfq4av32wahnp792adx4zba5aznv6ja7-rgbds-0.5.2/bin/rgbfix passed the binary check.
  • /nix/store/bfq4av32wahnp792adx4zba5aznv6ja7-rgbds-0.5.2/bin/rgbgfx passed the binary check.
  • 4 of 4 passed binary check by having a zero exit code.
  • 4 of 4 passed binary check by having the new version present in output.
  • found 0.5.2 with grep in /nix/store/bfq4av32wahnp792adx4zba5aznv6ja7-rgbds-0.5.2
  • found 0.5.2 in filename of file in /nix/store/bfq4av32wahnp792adx4zba5aznv6ja7-rgbds-0.5.2

Rebuild report (if merged into master) (click to expand)
5 total rebuild path(s)

3 package rebuild(s)

3 x86_64-linux rebuild(s)
2 x86_64-darwin rebuild(s)


First fifty rebuilds by attrpath
docbookrx
rgbds
sameboy
Instructions to test this update (click to expand)

Either download from Cachix:

nix-store -r /nix/store/bfq4av32wahnp792adx4zba5aznv6ja7-rgbds-0.5.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.

Or, build yourself:

nix-build -A rgbds https://github.com/r-ryantm/nixpkgs/archive/7b90792ac300afc703e57a56211928a81341b171.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:

ls -la /nix/store/bfq4av32wahnp792adx4zba5aznv6ja7-rgbds-0.5.2
ls -la /nix/store/bfq4av32wahnp792adx4zba5aznv6ja7-rgbds-0.5.2/bin


Pre-merge build results

We have automatically built all packages that will get rebuilt due to
this change.

This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.

Result of nixpkgs-review run on x86_64-linux 1

2 packages built:
  • rgbds
  • sameboy

Maintainer pings

cc @matthewbauer @NieDzejkob for testing.

@meithecatte
Copy link
Contributor

I don't think it's supposed to fail on Darwin. FWIW, it is a supported target upstream. Moreover, I don't see what could cause 0.5.x to fail on Darwin – I don't see any changes that could cause it in the git log for the Makefile.

I don't have a Mac I could use to debug this, but perhaps building with cmake instead could help?

@c0bw3b
Copy link
Contributor

c0bw3b commented Jan 2, 2022

Darwin builds for v0.5.1 are already failing on Hydra. I wanted to solve that along with the update.

But OK let's ask @NixOS/darwin-maintainers if someone can maybe build this locally and solve the linking issue?

ld: warning: ld: warning: ignoring file src/asm/charmap.o, building for macOS-x86_64 but attempting to link with file built for unknown-unsupported file format ( 0xDE 0xC0 0x17 0x0B 0x00 0x00 0x00 0x00 0x14 0x00 0x00 0x00 0x4C 0x28 0x00 0x00 )ignoring file src/asm/fixpoint.o, building for macOS-x86_64 but attempting to link with file built for unknown-unsupported file format ( 0xDE 0xC0 0x17 0x0B 0x00 0x00 0x00 0x00 0x14 0x00 0x00 0x00 0xA8 0x22 0x00 0x00 )

@meithecatte
Copy link
Contributor

meithecatte commented Jan 2, 2022 via email

@c0bw3b
Copy link
Contributor

c0bw3b commented Jan 3, 2022

Building with clangStdenv on Linux doesn't reproduce the issue.

@reckenrode
Copy link
Contributor

Seems to be the same problem as #19098. I added the following, and the derivation built on aarch64-darwin.

NIX_CFLAGS_COMPILE = lib.optional stdenv.isDarwin "-fno-lto";

@c0bw3b
Copy link
Contributor

c0bw3b commented Jan 3, 2022

Thanks @reckenrode
It adds up: upstream added -flto in v0.5.0 with commit gbdev/rgbds@b598911

@prusnak
Copy link
Member

prusnak commented Jan 3, 2022

@GrahamcOfBorg build rgbds

@c0bw3b
Copy link
Contributor

c0bw3b commented Jan 3, 2022

Last commit pushed already triggered ofBorg's eval and rebuilds @prusnak

Great it now builds on all platforms \o/

@c0bw3b c0bw3b merged commit ca485f4 into NixOS:master Jan 3, 2022
@r-ryantm r-ryantm deleted the auto-update/rgbds branch January 4, 2022 00:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants