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

fix: Make command execution on windows work for non-exe open methods #141

Merged
merged 1 commit into from
Aug 2, 2024

Conversation

ibotha
Copy link
Contributor

@ibotha ibotha commented Jul 10, 2024

This should make is so that using "start" as an open_program in the config actually works on windows. it changes the executable check out for an try-catch basically.

This should make is so that using "start" as an open_program in the
config actually works on windows. it changes the executable check out
for an try-catch basically.
@saecki
Copy link
Owner

saecki commented Jul 11, 2024

Does the vim.fn.executable check work if .exe isn't appended?
Maybe we could just change the is_binary_installed check instead (and possibly rename it), so that the health check doesn't fail...

@saecki
Copy link
Owner

saecki commented Aug 2, 2024

I don't think there is a way to check if start is executable and this seems to work...

@saecki saecki merged commit 1c924d5 into saecki:main Aug 2, 2024
2 checks passed
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.

2 participants