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

A problem with a previous commit ? #182

Open
turiot opened this issue Jan 5, 2022 · 4 comments
Open

A problem with a previous commit ? #182

turiot opened this issue Jan 5, 2022 · 4 comments

Comments

@turiot
Copy link

turiot commented Jan 5, 2022

Hello Oleg,

While recompiling trolCommander from develop branch I encounter an issue on Windows 10, the application can't open files.
It works when I revert commit 387e8ef ('Fixed command execution on Windows for path with whitespaces'), in WinNtDesktopAdapter.java, setting back FILE_OPENER_COMMAND to "cmd /c start "" "$f""

Is it my environnement, a lost commit, or something else ?

Thanks.

@trol73
Copy link
Owner

trol73 commented Jan 6, 2022

Hello,
Thanks for your message. As far as I remember, I made these changes because on my Windows 10 it was not possible to open a file located in a directory with spaces in the name. And this change solved the problem for me then.
But it turns out that the problem was not resolved at all..

@turiot
Copy link
Author

turiot commented Jan 7, 2022

Hi Oleg,
For me on windows 10 with the rollback, opening a file (launching a pdf external viewer, or an exe) works even with spaces in directory or file name.
Can I help with trying to recreate the same problematic configuration you got ?

@trol73
Copy link
Owner

trol73 commented Jan 10, 2022

Hi,
As far as I remember, I had a problem on Windows 10 when opening a file in the directory of which there was a space and Cyrillic characters. Like this "c:\Google Диск\some file.pdf"

@turiot
Copy link
Author

turiot commented Jan 11, 2022

привет,
I can't recreate the problem, on windows 10 + Java 16 + locale fr, a path like "c:\Google Диск\some file.pdf" works.

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

No branches or pull requests

2 participants