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
argtable: init at 2.13 #53007
argtable: init at 2.13 #53007
Conversation
version = "2.13"; | ||
|
||
src = fetchurl { | ||
url = http://prdownloads.sourceforge.net/argtable/argtable2-13.tar.gz; |
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.
mirror://sourceforge/argtable/argtable2-13.tar.gz
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.
You need to add this package in all-packages.nix
.
Could you also squash your commits please?
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.
Could you squash your commits also?
@@ -577,6 +577,8 @@ in | |||
|
|||
argtable = callPackage ../tools/misc/argtable {}; |
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.
I think it would be nice to rename the existing argtable
to argtable3
(both attribute and directory names).
And also define
argtable = argtable3;
It seems argtable3 (release 2017) is well maintained then argtable2 (release 2011) so we should encourage users to use argtable3. What do you think?
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.
👍 is it possible to change the argtable
directory name - or are there other implications ?
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.
Only the attribute name is used by other derivations. So, you can change the directory path.
Are there any updates on this pull request, please? |
Thank you for your contributions.
|
The last upstream news: |
Motivation for this change
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)