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

Update installation.md #199

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

vineelkovvuri
Copy link

On Windows, YAZI_FILE_ONE environment variable should be specified without double quotes. With double quotes the variable is processed as below. Resulting in file preview not working.

YAZI_FILE_ONE : Some(""C:\Program Files\Git\usr\bin\file.exe"")
file : The filename, directory name, or volume label syntax is incorrect. (os error 123)

On Windows, YAZI_FILE_ONE environment variable should be specified without double quotes. With double quotes the variable is processed as below

YAZI_FILE_ONE   : Some("\"C:\\Program Files\\Git\\usr\\bin\\file.exe\"")

and results in 
file          : The filename, directory name, or volume label syntax is incorrect. (os error 123)
@sxyazi
Copy link
Member

sxyazi commented Jan 20, 2025

I think it depends on how the user has set it up. If it's set in the system environment variables, then no quotes are needed, but if it's set in the PowerShell configuration file, then quotes are required.

@vineelkovvuri
Copy link
Author

Good point. Instead of the current change, should I add a note that the double quotes should not be provided when setting the environment variable via Windows system UI(sysdm.cpl). Or better let me take a shot at fixing the issue in the code.

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