Skip to content

stack path: Rename --ghc-paths to --programs-path, add --compiler-path #1546#2016

Merged
mgsloan merged 3 commits into
commercialhaskell:masterfrom
sjakobi:1546-ghc-path
Apr 11, 2016
Merged

stack path: Rename --ghc-paths to --programs-path, add --compiler-path #1546#2016
mgsloan merged 3 commits into
commercialhaskell:masterfrom
sjakobi:1546-ghc-path

Conversation

@sjakobi
Copy link
Copy Markdown
Member

@sjakobi sjakobi commented Apr 9, 2016

No description provided.

@sjakobi sjakobi changed the title stack path: Remove --ghc-paths, add --compiler-path stack path: Remove --ghc-paths, add --compiler-path #1546 Apr 9, 2016
@sjakobi
Copy link
Copy Markdown
Member Author

sjakobi commented Apr 9, 2016

There's another reference to --ghc-paths in the guide where I'm not quite sure how to update it because I don't have a Windows system:

On Windows, you will also need to delete stack path --ghc-paths

@mgsloan
Copy link
Copy Markdown
Contributor

mgsloan commented Apr 11, 2016

I think that comment should be updated with --programs-path (and an implementation of it). I suppose this is the usecase for the existence of such a flag. The issue is that on windows, the programs path goes into some appdata dir in the user folder, or something like that.

@sjakobi
Copy link
Copy Markdown
Member Author

sjakobi commented Apr 11, 2016

I think that comment should be updated with --programs-path (and an implementation of it).

Done!

@sjakobi sjakobi changed the title stack path: Remove --ghc-paths, add --compiler-path #1546 stack path: Rename --ghc-paths to --programs-path, add --compiler-path #1546 Apr 11, 2016
@mgsloan
Copy link
Copy Markdown
Contributor

mgsloan commented Apr 11, 2016

Thanks!!

@mgsloan mgsloan merged commit 2e7e684 into commercialhaskell:master Apr 11, 2016
@sjakobi sjakobi deleted the 1546-ghc-path branch April 11, 2016 22:32
@mgsloan
Copy link
Copy Markdown
Contributor

mgsloan commented Apr 13, 2016

FYI, I've made a few updates that bikeshed this a bit, while it's easy to change: 2f7755d

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.

3 participants