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
acme-client: 0.2.5 -> 1.0.1 #88201
acme-client: 0.2.5 -> 1.0.1 #88201
Conversation
The upstream version fails to compile due to a missing limits.h include. I added a patch to fix that. I opened a pull request to upstream it too, but the project has moved from GitHub onto sr.ht and now asks me to send a patch to the mailing list. My default email client is not really suitable for that, and getting git-send-email set up will take some work, so in the meantime it is easier to just patch it here.
1.0.1 is out and includes your patch. I would also suggest using tar balls from https://data.wolfsden.cz/sources/, it will allow you to skip the |
I submitted my patch upstream, it was merged, and version 1.0.1 which includes it has been released. So the patch here is no longer necessary. Also, the maintainer added a Nixpkgs-based build environment to the upstream CI setup, so in the future the upstream version will likely not need any patching.
This was suggested by the upstream maintainer [1], and it is a nice simplification. Also change the url to sr.ht as the project has moved there. [1]: NixOS#88201 (comment)
Thanks for the quick release! I updated this pull request.
I did not know about these sources, thanks for the suggestion. That is a nice simplification of the package, I added that to this PR. @pmahoney, could you please take a look? |
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: |
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 good.
@graywolf is that URL preferable to, e.g. https://git.sr.ht/~graywolf/acme-client-portable/archive/v1.0.1.tar.gz ? Is it documented somewhere other than your comment above?
In general yes, it is preferable, since it is output of |
Motivation for this change
A new version has been released upstream.
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)Initially this pull request was for version 1.0.0, which needed a small patch for a missing include. This has now been fixed upstream in version 1.0.1, and I have updated this pull request accordingly.