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
pyexiv2: 0.3.2 -> 2.1.0 #82771
pyexiv2: 0.3.2 -> 2.1.0 #82771
Conversation
Thanks for the feedback. All suggestions are applied and everything seems to be in working order. |
@GrahamcOfBorg build pyexiv2 |
@GrahamcOfBorg build python3Packages.pyexiv2 |
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.
Ran 0 tests in 0.000s
No tests are found. Either the test runner cannot find tests and needs to be patched, or there simply are no tests in which case the tests need to be disabled. Do include a comment explaining why the tests are disabled.
I took a look at what was going on. I'll admit I'm not 100% sure what the deal is because I am not overly familiar with how setuptools integrates with stuff like pytest, however, I suspect that it was not able to resolve the tests because its in a non-standard location ( |
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.
otherwise LGTM, tests run
pushd pyexiv2/tests | ||
pytest | ||
popd |
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.
pushd pyexiv2/tests | |
pytest | |
popd | |
pytest pyexiv2/tests |
I marked this as stale due to inactivity. → More info |
Please rebase |
Closing since pyexiv2 has been deleted from nixpkgs in #141794. Feel free to create another one to add it again. |
Motivation for this change
pyexiv2 has been broken for quite a while, but since then it has been rewritten. It no longer has a Boost dependency and now only works on Python 3 (formerly, it only worked on Python 2.)
Only one package depends on pyexiv2, jbrout. However, it is old and broken, so I made a PR to remove it in #82770.
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)