Skip to content
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

Allow to specify a build config for 'ghcup compile' #20

Closed
wants to merge 1 commit into from

Conversation

hasufell
Copy link
Member

Fixes #19

@hasufell hasufell requested a review from hvr October 16, 2018 08:19
@hasufell hasufell force-pushed the PR/allow-to-specify-build-config branch from c8d2e5a to ecbd47c Compare October 16, 2018 09:28
@hasufell hasufell changed the title Allow to specifc a build config for 'ghcup compile' Allow to specify a build config for 'ghcup compile' Oct 16, 2018
Copy link
Member

@hvr hvr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I briefly tried this on a deb8 system and it worked fine

-f, --force Overwrite already existing installation
-j, --jobs <n> How many jobs for compilation
-c, --build-config <filepath> Use the given config file as build config
(mk/build.mk)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Btw, at first, the "(mk/buiild.mk)" seemed a bit confusing to me, as I wrongly assumed that'd be the default <filepath>

edo cat "${build_config}" > mk/build.mk
else
cat <<-EOF > mk/build.mk || die
BuildFlavour = quick
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

btw, BuildFlavour = quick won't have any effect, that's not how that var works... (nor would it be a good idea to use that build-profile even if it worked)

@hasufell
Copy link
Member Author

merged and fixed as 69e2060

@hasufell hasufell closed this Oct 17, 2018
@hasufell hasufell deleted the PR/allow-to-specify-build-config branch October 17, 2018 13:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants