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
pkgconf: init at 1.5.4 #50771
pkgconf: init at 1.5.4 #50771
Conversation
|
||
meta = with stdenv.lib; { | ||
description = "Package compiler and linker metadata toolkit"; | ||
homepage = https://git.dereferenced.org/pkgconf/pkgconf; |
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.
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.
The website is clearly out-of-date and returns an invalid HTTPS certificate. I would recommend against linking it in the derivation. It also points to GitHub, which is not the official repository any longer.
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.
Any more concerns on this? The rest should be fixed.
Could you try reporting the broken website in their issue tracker?
…On Tue, 20 Nov 2018, 10:46 Francesco Zanini ***@***.*** wrote:
***@***.**** commented on this pull request.
------------------------------
In pkgs/development/tools/misc/pkgconf/default.nix
<#50771 (comment)>:
> + url = "https://git.dereferenced.org/pkgconf/pkgconf.git";
+ rev = "74133eda31bc1ed5947b4a3a854001e320b6c1fe";
+ sha256 = "159fxbwm5shz8p95jp28wrjvinlhmp42dy60pqg34psjn41wq1q4";
+ };
+
+ buildInputs = [ automake autoconf libtool ];
+
+ preConfigurePhases = ["autogenPhase"];
+
+ autogenPhase = ''
+ ./autogen.sh
+ '';
+
+ meta = with stdenv.lib; {
+ description = "Package compiler and linker metadata toolkit";
+ homepage = https://git.dereferenced.org/pkgconf/pkgconf;
Any more concerns on this? The rest should be fixed.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#50771 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AArCYzse3eXNEuj1x5ml4LWe-4rDiLG1ks5uw89lgaJpZM4YpsVS>
.
|
https://git.dereferenced.org/pkgconf/pkgconf/issues/16 I will follow up on this if any news comes by. Should this PR be merged anyway, and I submit a new one when / if further information is available? |
@GrahamcOfBorg build pkgconf |
Success on x86_64-linux (full log) Attempted: pkgconf Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: pkgconf Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: pkgconf Partial log (click to expand)
|
Thanks. we will fix the website when they respond. |
Motivation for this change
Add a new derivation.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)