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
patchutils: 0.3.3 -> 0.3.4, keep 0.3.3 for fetchpatch #59422
Conversation
@GrahamcOfBorg build patchutils patchutils_0_3_3 |
Looking into the earlier problem with |
I tested this with
This seems safe to use now for |
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.
Code seems good. I also checked fetchpatch
on a few places (like GitHub).
Motivation for this change
See #38618
Closes #38618
This upgrades patchutils from 0.3.3 -> 0.3.4. However, there were issues with its usage from
fetchpatch
combined with patchutils 0.3.4. One of the possible solutions that was raised by @dezgeg was to add both versions of patchutils. That is what I have attempted to to in this PR.To test whether
fetchpatch
still works:Tried the following to see whether both versions of patchutils are working:
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)