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
python3Packages.pyelftools: disable tests on Darwin #100602
Conversation
f9b3cbb
to
9c5cf87
Compare
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.
LGTM 👍
Builds properly on NixOS.
e6b499c
to
3910fe4
Compare
3910fe4
to
183ffad
Compare
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.
LGTM
af27072 broke the build on Darwin. There does not seem to be any way to sanely enable tests on Darwin. Tests initially fail because of missing 'greadelf' expected by test/run_readelf_tests.py. This is fixed by: - adding 'binutils-unwrapped' to checkInputs - hardlinking 'readelf' to 'greadelf' in a temp bin dir - exporting temp bin dir to PATH This leads to further test failures because of 'readelf' version mismatch causing output inconsistencies. See test/run_readelf_tests.py:31 for further details. Signed-off-by: Sirio Balmelli <sirio@b-ad.ch>
183ffad
to
080e6ca
Compare
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.
LGTM
Result of nixpkgs-review pr 100602 1
af27072 broke the build on Darwin (see #99202)
There does not seem to be any way to sanely enable tests on Darwin.
Tests initially fail because of missing 'greadelf'
expected by test/run_readelf_tests.py.
This is fixed by:
This leads to further test failures because of 'readelf' version mismatch
causing output inconsistencies.
See test/run_readelf_tests.py:31 for further details.
Signed-off-by: Sirio Balmelli sirio@b-ad.ch
Motivation for this change
build on Darwin
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)Ensured that relevant documentation is up to date