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
voxelands: 1512.00 -> 1704.00 #104705
voxelands: 1512.00 -> 1704.00 #104705
Conversation
pkgs/games/voxelands/default.nix
Outdated
cmake irrlicht libpng bzip2 libjpeg sqlite | ||
libXxf86vm libGLU libGL openal libvorbis xlibsWrapper pkgconfig | ||
bzip2 | ||
cmake |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cmake and pkg-config should be in nativeBuildInputs
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Addressed in 45f32f9e82b79d49c1b11e1c4630463cbde14168
It fails on aarch64 https://logs.nix.ci/?key=nixos/nixpkgs.104705&attempt_id=ce623944-0fb0-4184-8afb-3211576e4a57 Perhaps it should be marked as broken for this system? |
This seems more like a issue with Irrlicht engine on aarch64 - see https://sourceforge.net/p/irrlicht/bugs/443/ (irrlich thinks the system libpng was compiled with NEON instructions support, but it seems it's not) |
Is irrlicht a bundled dependency built within the build of voxeland? Because from the log it seems it's voxeland's build that fails: https://logs.nix.ci/?key=nixos/nixpkgs.104705&attempt_id=ce623944-0fb0-4184-8afb-3211576e4a57 |
No. If I see it correctly voxeland uses system irrlicht. |
Then if irrlicht was the failing build, the build log wouldn't have looked like this:
As you can see, it's voxelands' binaries that fail to link, not irrlicht's. Whatever the reason is, we should mark it as broken for aarch64, ideally with a comment saying the build expects neon symbols or something like that. |
Addressed in 1ed49e3 |
Motivation for this change
Version update, Repo change URL change
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)