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

Windows fixes #74

Merged
merged 4 commits into from
May 14, 2024
Merged

Windows fixes #74

merged 4 commits into from
May 14, 2024

Conversation

mikavilpas
Copy link
Owner

Continuing the work started in #73. Got stalled because of build system issues 🤷🏻‍♂️

@mikavilpas
Copy link
Owner Author

@Kayzels I noticed I have broken a feature on osx (the platform I use right now). I will fix it for me, and maybe you could try and see if the windows fixes need some change after that.

when nvim was editing a file with spaces in its name, it would not be
preselected as the current file when opening yazi. This seems like a
regression.
@Kayzels
Copy link
Contributor

Kayzels commented May 14, 2024

Doesn't this last commit break the changes for 051bfce, though? I will test it soon.

@mikavilpas
Copy link
Owner Author

Yeah I think it did break a bunch of stuff, but it seemed to work now that I tried. I adapted the tests to what seemed to work for me in practice.

@Kayzels
Copy link
Contributor

Kayzels commented May 14, 2024

So, I'll need to test it now. From what I can tell, you removed the quotes around the input file for Yazi? With that, now the spaces will need to be escaped, so my earlier commit probably won't work. I'll see if there's a way to get it working.

@Kayzels
Copy link
Contributor

Kayzels commented May 14, 2024

There isn't an easy way to escape spaces in Powershell, so this removal of the quotes is a breaking change. Is there no way to keep them in while still working on Mac? Unfortunately, the tests aren't system agnostic, so changes to make it work for Windows will cause some tests to fail, from what I can see.

@mikavilpas
Copy link
Owner Author

Hmm, I think it's possible to keep them on Windows specifically. I pushed a change to hopefully fix this for you.

The testing setup doesn't work at all on windows right now 😅

Can you try the latest version out?

@mikavilpas mikavilpas merged commit 1dd8403 into master May 14, 2024
14 checks passed
@mikavilpas mikavilpas deleted the windows-fix branch May 14, 2024 18:36
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