Skip to content
This repository has been archived by the owner on Oct 7, 2020. It is now read-only.

Add installation instructions for windows users #455

Merged
merged 1 commit into from
Jan 30, 2018
Merged

Add installation instructions for windows users #455

merged 1 commit into from
Jan 30, 2018

Conversation

ShrykeWindgrace
Copy link

Add installation instructions for windows users who are facing the path too long issue on windows. Addresses #435

@alanz
Copy link
Collaborator

alanz commented Jan 30, 2018

Thanks. Every bit helps

@alanz alanz merged commit 3787093 into haskell:master Jan 30, 2018
@nponeccop
Copy link
Contributor

nponeccop commented Jan 30, 2018

@alanz apparently it's not #435 but an unrelated issue. And now we have an instruction suggesting fairly terrible things to everyone on Windows. While in fact it's not for everyone but for those using " haskell-ide-engine somewhere deep in folder hierarchy" as @ShrykeWindgrace put it in #435.

The practice of creating anything out of home folder is as terrible on Windows as it is on Linux (although historically nobody cares - the tradition does back all the way to DOS and it's predecessors). "Normal" folders complying with Windows analog of Filesystem Hierarchy Standard such as c:\users\username\Documents\dev\foo\bar\quux\haskell-ide-engine work just fine, the path just cannot be too long. So the installation instruction should explicitly mention that it's a workaround for a rarely encountered issue.

@alanz
Copy link
Collaborator

alanz commented Jan 30, 2018

Ok, so what to do?

@nponeccop
Copy link
Contributor

the installation instruction should explicitly mention that it's a workaround for a rarely encountered issue.

Either I or @ShrykeWindgrace will change the instruction text when we have time. Hopefully tomorrow.

@ShrykeWindgrace
Copy link
Author

On a second thought, I'd say it's better to revert the merge altogether. The workaround is windows-centric and has little to do with hie itself.

@alanz alanz added this to the prehistory milestone Feb 2, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants