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

[ Feature Request ] Windows support #236

Open
j-nava opened this issue Mar 5, 2023 · 2 comments
Open

[ Feature Request ] Windows support #236

j-nava opened this issue Mar 5, 2023 · 2 comments
Labels
documentation Improvements or additions to documentation problem Something isn't working

Comments

@j-nava
Copy link

j-nava commented Mar 5, 2023

The installation script works just fine in MSYS2, which is a great step forward, but pack itself fails with the message "[ fatal ] Failed to figure out package directory."

The problem seems to be the filepath library, which only supports Linux-style paths when parsing.
Even in MSYS2, environment variables are unfortunately separated by "\", so I believe there's currently no workaround to get pack working.

Is Windows support something we would like to pursue? If so, what would be a good first step? Modifying filepath, adding a new cmd argument to pack, or other solution?

@stefan-hoeck
Copy link
Owner

stefan-hoeck commented Mar 28, 2023

Sorry for the late reply. Obviously, I'm not a Windows users, so I won't myself be able to test / maintain Windows support. This means that if Windows support breaks and nobody is around to fix it, I'll have to drop it. But I'll definitely accept contributions in this regard.

Currently, I think there are two issues that need to be addressed: First, as you correctly wrote, the idris2-filepath library currently only supports Unix-style file paths. Contributions there are very welcome. Second, many IO operations in Pack.Core.IO (and possibly in Pack.Core.Git) are implemented as syscalls to typical Unix utilities. These would need to be changed in such a way that the syscalls being made work correctly on Windows (if we are on a Windows system).

Finally, I'll only accept contributions together with an update to the docs and an inclusion in our CI tests.

Edit: That being said, I'll of course help with setting up this stuff and figuring the places in the code that need to be adjusted.

@isberg
Copy link

isberg commented Jun 30, 2024

For anyone else struggling on Windows: I solved it by using docker and base image ghcr.io/stefan-hoeck/idris2-pack. I then created my own image where I installed neovim and let pack install idris2-lsp.

@buzden buzden added problem Something isn't working documentation Improvements or additions to documentation labels Jul 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation problem Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants