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
miniupnpc_2: 2.1 -> 2.1.20190408, miniupnpd: 2.1 -> 2.1.20190502 #61363
Conversation
@GrahamcOfBorg build altcoins.bitcoind altcoins.wownero |
@GrahamcOfBorg test upnp |
Looking at the repository of miniupnpc I couldn't find a reason why there was no release (new version) yet. Any ideas? |
So now I noticed I don't know the different between the 1.x series and 2.x series. I assume a project rewrite or some such? It probably makes sense to eventually drop the 1.x series... if anyone is up to the task of ensuring everything still works on the 2.x series. |
I'm basing it off the official website http://miniupnp.free.fr/ on the download page. Unless the website were to explicitly state github is the official source I would continue to use the main website as the most up to date source I guess... but yes I agree it is odd the github page doesn't have that release. |
Aah, I see. GitHub tags/releases aren't used. For anyone else, the changelog can be found here: http://miniupnp.free.fr/files/changelog.php?file=miniupnpd-2.1.20190502.tar.gz Thanks for the information. This looks good 👍. |
Motivation for this change
#57320
@bobvanderlinden as the listed maintainer of the nixos upnp test do you know any reason that we should maintain the miniupnpc 1.x package from 2016? If you can't think of any reasons I'll replace my last commit with a commit to drop the 1.x package.
ping @dtzWill as potential interested party because of comments on #57320
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)