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
gnuapl: 1.6 -> 1.7 #24114
gnuapl: 1.6 -> 1.7 #24114
Conversation
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.
Remove with stdenv.lib; from meta and move to top then you can then remove stdenv.libbfrom optionals and optionalString
@ndowens What is the rationale for such a refactoring? As it stands, this update looks good to me. |
It is just cosmetic thing I like to do, mostly just to use stdenv.lib once vs several times |
I have no problem doing the refactoring. I’ll to that tonight. |
I don't know why wait to do another PR just for cosmetic change?
…Sent from my iphone
On Mar 21, 2017, 5:28 AM -0500, Matthew Daiter ***@***.***>, wrote:
@lsix (https://github.com/lsix) @ndowens (https://github.com/ndowens) should we leave refactoring to another PR?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub (#24114 (comment)), or mute the thread (https://github.com/notifications/unsubscribe-auth/AAHL753Qk6YyaxCIiq-ew498grq0KM--ks5rn6ZMgaJpZM4MiI2-).
|
@ndowens it primarily makes bookkeeping easier. |
Idk. If you change it and commit with msg cosmetic change or so that is bookkeeping IMO and can be split to fix something if it had to be
…Sent from my iphone
On Mar 21, 2017, 6:10 AM -0500, Matthew Daiter ***@***.***>, wrote:
@ndowens (https://github.com/ndowens) it primarily makes bookkeeping easier.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub (#24114 (comment)), or mute the thread (https://github.com/notifications/unsubscribe-auth/AAHL78aOl_oVZ5YAIqU_m_4wkT4XSJodks5rn7AZgaJpZM4MiI2-).
|
|
||
src = fetchurl { | ||
url = "mirror://gnu/apl/apl-${version}.tar.gz"; | ||
sha256 = "057zwzvvgcrrwsl52a27w86hgy31jqq6avqq629xj7yq90qah3ay"; | ||
sha256 = "07xq8ddlmz8psvsmwr23gar108ri0lwmw0n6kpxcv8ypas1f5xlg"; | ||
}; | ||
|
||
buildInputs = [ readline gettext ncurses ]; |
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.
"--disable-dependency-tracking" is added by default to configure.
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 also do not see strong reason to keep the "--disable-silent-rules"
(darwin only, but default behavior of automake). I’m going to drop it all together.
See http://lists.gnu.org/archive/html/info-gnu/2017-03/msg00007.html for release announcement.
See http://lists.gnu.org/archive/html/info-gnu/2017-03/msg00007.html
for release announcement.
Motivation for this change
Things done
(nix.useSandbox on NixOS,
or option
build-use-sandbox
innix.conf
on non-NixOS)
nix-shell -p nox --run "nox-review wip"
./result/bin/
)