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.pytest-timeout: update patch URL #111268
Conversation
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.
looks like they moved https://github.com/pytest-dev/pytest-timeout
but I'm okay with vendoring the patch.
However, the patch doesn't seem nixos specific. Is there a reason why it hasn't been merged yet?
It appears to have been merged in newer versions of pytest-timeout, for example and the patch is no longer specified in nixpkgs master which uses the newer version. I figured adding the patch directly was less intrusive to 20.09 then bumping the version to follow master though. |
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.
Just replace the URL with https://github.com/pytest-dev/pytest-timeout/commit/20c2ba5ef60007b5071acfd62b326cafce0e5493.patch. Then you can even target release-20.09
.
44b1612
to
733c4f3
Compare
Ok. I've switch to just updating the URL and rebased on release-20.09. Thanks! PS: How did you find the commit on the github side? (I had thought the bitbucket URL contained the commit hash, and when I couldn't find that hash, I had assumed it didn't exist) |
I just went through the history and found a matching commit message. |
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.
You need to change the commit message.
Also, we apparently need to add name = "raw"
in order to make this not cause any rebuilds. Maybe you can add a comment with the reason for adding the line.
733c4f3
to
326d6d7
Compare
Thanks. Done and done. 👍 |
I don't mind if this causes any rebuilds and we would clean it up a bit. Python builds really fast anyway. |
Motivation for this change
Patch URL is invalid and couldn't find another, so directly included it (I've now seen several patches with sour URLs, why not just include it in the first place and the potential for this pain?).
Things done
Add the patch directly.
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)