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

Interpreter: Provide a way to hide implicit packages #2480

Merged
merged 1 commit into from
Aug 13, 2016

Conversation

harendra-kumar
Copy link
Collaborator

Always pass all the explicitly specified packages in the script to ghc command
line. Also allow ghc options to be specified from the interpreter command. This
will give user the flixibility to, for example, use '-hide-all-packages' ghc
option. Like this:

{- stack runghc
--package base
--package exceptions
--package transformers
--package containers
--package getopt-generics
--package filepath
--package path
--package path-io
--package process
--package unix-compat
--
-hide-all-packages
-}

fixes #1208

Always pass all the explicitly specified packages in the script to ghc command
line. Also allow ghc options to be specified from the interpreter command. This
will give user the flixibility to, for example, use '-hide-all-packages' ghc
option. Like this:

{- stack runghc
    --package base
    --package exceptions
    --package transformers
    --package containers
    --package getopt-generics
    --package filepath
    --package path
    --package path-io
    --package process
    --package unix-compat
    --
    -hide-all-packages
-}

fixes #1208
@harendra-kumar
Copy link
Collaborator Author

Windows build failing due to #2457 fixed by #2482 . I will rebase and try again.

@Blaisorblade
Copy link
Collaborator

LGTM!

@Blaisorblade Blaisorblade merged commit 54f59bd into master Aug 13, 2016
@Blaisorblade Blaisorblade deleted the self-contained-scripts branch August 13, 2016 10:44
@Blaisorblade
Copy link
Collaborator

@harendra-kumar thanks for this. Maybe you can add a mention for this in the docs?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

interpreter: Don't make cached packages available that are not specified with --package
4 participants