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
Protonvpn ng update 2.2.0 to 2.2.2 #81735
Protonvpn ng update 2.2.0 to 2.2.2 #81735
Conversation
Built and tested locally, it works fine. Could you please merge your two commits into one? |
@jtcoolen Is there a reason you would prefer to have it into one commit? is it for the backport? I can take care of that. |
The second change is really part of the first one, so one commit suffices. |
57135ab
to
eb96574
Compare
@jtcoolen done! |
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.
Perfect! Thanks for contributing! Could you merge this change @worldofpeace?
@jtcoolen I will take care of backporting this to 19.09 and 20.03 later today! Thank you for your contributions as well my dear fellow Nix User :) |
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/pr-approved-needs-merging-protonvpn-cli-ng/6406/1 |
I have found that we were missing one dependency with this package: On a new installation, I wasn't able to connect to one of the VPNs, until I installed |
eb96574
to
5cb4488
Compare
LGTM, can you squash the two commits, then I'll merge? |
5cb4488
to
9fe4a63
Compare
@bhipple Done! I will backport the commit to the 20.03 and 19.09! :D Thank you |
|
Motivation for this change
I use this VPN and I was getting messages telling me to upgrade. I wanted to give this to the community
Things done
Update version from 2.2.0 to 2.2.2, update the sha256 sum, and add myself as a maintainer of this package.
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)