-
Notifications
You must be signed in to change notification settings - Fork 18
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
Wrong path name when installing packages on Git Bash for Windows(zsh shell) #121
Comments
Git Bash is, by default, Bash. I hate to ask, but can you confirm you are you running Zsh from bash before running antidote, or are you just hoping antidote will work in straight git bash? Antidote is, and will always be, a Zsh-only plugin manager, so I just need to confirm. |
I'm open to see if we can get things working in git bash. I just don't have a Windows machine handy at the moment, so it'll take me a bit to get a test scenario set up to see if I can reproduce this. If so, and it's something that can be addressed within antidote, I'll consider coding a fix. |
Without any promises I am able to use windows and have some experience with shell programming. I am on finals but I am pretty sure this is an easy fix. I'll see if I can fix the issue and submit a pr :) |
One question to speed things up. Per my understanding it seems like the path used when antidote generates It would be helpful to know this information, as I can just make it point to the right location when on Mingw64 |
This is the function that determines the home: https://github.com/mattmc3/antidote/blob/main/functions/antidote-home |
Submited a PR! |
@mattmc3 I just tried to set it up again and am getting the same issue: In this case, I just tried to utilize the zdotdir project as a template. Not sure if something needs to be done there? EDIT: did some more experimentation.. and it appears to work fine within git bash but if you're trying to use git bash within vscode, it does not work. |
@benwilburn can you try using my fork and see if that works? |
I'm also having the same issue as @benwilburn when trying to use antidote in zsh invoked from git bash. If I add the
Then the issue is fixed and antidote works correctly. Could the fix by @DiegoMagdaleno be merged in @mattmc3? |
I can't merge that in without a good way to test it into the future, so what I'm really held up by is making sure there's a CI workflow to verify this works. I don't run Windows personally, so I have no way to see that this is working without that workflow, which wasn't part of the PR. |
There may be a way to have the GitHub runners handle this, since they do come with git bash installed by default. @mattmc3, if I figure out a way to hook it into a Windows runner, would you be open to a PR? |
Yes, definitely. |
I can take full responsability for maintaining all of the Windows related bugs. If you wish :) |
If v1.9.2 doesn't fix it, please re-open. I was able to properly test on an old Windows clunker, so I'm reasonably confident I can support this without CI working for Windows yet. No reason to hold up the feature. Thanks for the |
When attempting to install packages via antidote in Git Bash for Windows, it fails to install packages due to path errors (seems to work fine in WSL). It would be nice to be able to use Antidote in Git Bash
The text was updated successfully, but these errors were encountered: