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
pythonPackages.gcovr: Fix build #56851
Conversation
@GrahamcOfBorg build python2Packages.gcovr python3Packages.gcovr |
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.
Is this even used as python module?
Could you also use the |
@worldofpeace Done. Also added an alias to use the package simply as |
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.
Didn't see any tests being ran so they should be disabled
doCheck = false;
Or change the source to github and try to enable them.
ping @knedlsepp |
Since gcovr is most likely to be used as a script instead of a library, we provide an alias to python3Packages.gcovr. We still keep python2/3Packages.gcovr in case somebody really wants to import it as a library, which can not entirely be ruled out.
@worldofpeace Sorry for the late reply. Did fixup the suggestions that you made. I tried switching to the github source for the tests, finally could make it work by disabling hardening flags and using gcc5 + some manual patching. Since the tests highly depend on the gcc version I figured it wasn't really worth the maintenance effort on our side. |
Indeed that is true. We can wait for upstream for that. Thanks for completing this ✨ |
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.
see above comment
Thx! |
Motivation for this change
This fixes pythonPackages.gcovr. Please backport to 19.03.
ZHF-19.03: #56826
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)