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
asymptote: 2.47 -> 2.60 #73474
asymptote: 2.47 -> 2.60 #73474
Conversation
93eeb39
to
b4ef360
Compare
autoconf | ||
bison | ||
flex | ||
yacc |
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.
try adding autoreconfHook
to nativeBuildInputs
instead. All non-runtime dependencies should go there.
Is tex required at runtime or does it only build the docs? Then move it to nativeBuildInputs
, too.
rm *.tar.gz | ||
configureFlags="$configureFlags --with-latex=$out/share/texmf/tex/latex --with-context=$out/share/texmf/tex/context/third" | ||
HOME=$TMP | ||
./autogen.sh |
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.
With autoreconfHook
you'll be able to get rid of this.
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.
Removing HOME=$TMP
emits a bunch issues related to homeless-shelter
so I've kept that in.
configureFlags="$configureFlags \ | ||
--with-latex=$out/share/texmf/tex/latex \ | ||
--with-context=$out/share/texmf/tex/context/third" | ||
''; |
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.
Suggestion
- configureFlags="$configureFlags \
- --with-latex=$out/share/texmf/tex/latex \
- --with-context=$out/share/texmf/tex/context/third"
- '';
+'';
+configureFlags = [
+ "--with-latex=$out/share/texmf/tex/latex"
+ "--with-context=$out/share/texmf/tex/context/third"
+];
mkdir -p $out/share/emacs/site-lisp/${name} | ||
mv $out/share/asymptote/*.el $out/share/emacs/site-lisp/${name} |
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.
Should be possible with install as a single command.
''; | ||
|
||
enableParallelBuilding = true; | ||
|
||
meta = with stdenv.lib; { | ||
inherit (s) version; | ||
version = "${version}"; |
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.
version = "${version}"; |
Not really needed, not sure why it was there before.
b4ef360
to
2c40a74
Compare
@JohnAZoidberg thanks for the very helpful review! I've made most of the changes and included a few comments next to your suggestions. |
Also cleans up some unnecessary steps in the build expression.
2c40a74
to
098a903
Compare
Builds an runs at least the help command fine, which opens the PDF help document. |
Motivation for this change
Among other things, v2.6 improves usability for luatex users.
Things done
I've bumped asymptote to version 2.60. I've also removed some parts of the build expression that are no longer necessary and reorganized some other parts.
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @