Skip to content

Update HIE install instructions #176

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

Merged
merged 1 commit into from
Nov 20, 2019
Merged

Update HIE install instructions #176

merged 1 commit into from
Nov 20, 2019

Conversation

Gipphe
Copy link
Contributor

@Gipphe Gipphe commented Nov 20, 2019

Recommend ./install.hs build instead of the now removed ./install.hs build-all.

Add section on installing on Windows.

Closes #175

@alanz
Copy link
Collaborator

alanz commented Nov 20, 2019

@jneira are you happy with this?

@nponeccop
Copy link

nponeccop commented Nov 20, 2019

I think we should universally recommend enabling per-user Unicode paths for Git. I don't see any downsides. And the Unicode paths is basically using the right API instead of a legacy API. GHC for example already does that. If the right API is used, there are no limits for paths, except for SetCurrentDirectory calls, which (as Microsoft explains that) cannot be modernized because GetCurrentDirectory doesn't work well in multithreaded setting, so making it

Currently it looks like the Git setting and Long Paths setting are interrelated while they are not. The former is fixing a bug in Git while the latter is sort of workaround which breaks behaviour system-wide and it's not always a safe thing to do.

I'm not sure if we should have a separate issue/PR for this - there has been too many Readme-related changes lately.

@jneira
Copy link
Member

jneira commented Nov 20, 2019

are you happy with this?

yeah, thanks for updating this @Gipphe

@alanz alanz merged commit 9c6a603 into haskell:master Nov 20, 2019
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.

Update HIE install instructions
4 participants