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
glulxe: init at 0.5.4 #60302
glulxe: init at 0.5.4 #60302
Conversation
--replace "#GLKMAKEFILE = Make.glkterm" "GLKMAKEFILE = Make.glkterm" | ||
|
||
cd ../glkterm | ||
make |
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.
Why is make invoked in the configure phase? I suppose you want to invoke it in the build phase? In that case, use preBuild
and postBuild
to change folder, and keep the default buildPhase
, to ensure make
gets invoked with makeFlags
etc.
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.
Annoyingly to me as well, the Makefile in glkterm is acting more like "configure" than a build: it is producing the Make.glkterm which is then used by the build of glulxe.
--replace "#GLKMAKEFILE = Make.glkterm" "GLKMAKEFILE = Make.glkterm" | ||
|
||
cd ../glkterm | ||
make |
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 can use make -C dir
to execute the Makefile in an arbitrary subdirectory. That gets rid of the jumping in and out of directories.
Thank you for your contributions.
|
Motivation for this change
glulxe is available in guix but not in nixpkgs
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)