Extras icon indicating copy to clipboard operation
Extras copied to clipboard

windows-terminal: fix context menu error in drive root folder

Open AkariiinMKII opened this issue 2 years ago • 8 comments

Closes #8689

AkariiinMKII avatar Jun 14 '22 17:06 AkariiinMKII

/verify

AkariiinMKII avatar Jun 14 '22 17:06 AkariiinMKII

How has this been tested?

rashil2000 avatar Jun 14 '22 18:06 rashil2000

Could the powershell gallery being down be causing this?

edit: this being the ci failure.

chadcatlett avatar Jun 14 '22 19:06 chadcatlett

How has this been tested?

Well, here is an example: While in disk volume C, the %V returns C:\, and the command being "wt.exe" -d "C:\", somehow the backslash is treated as a escape character in this situation, then wt returns error could not access starting directory "C:"" Fortunately C:/ is a valid path for wt

AkariiinMKII avatar Jun 15 '22 00:06 AkariiinMKII

How has this been tested?

Well, here is an example: While in disk volume C, the %V returns C:\, and the command being "wt.exe" -d "C:\", somehow the backslash is treated as a escape character in this situation, then wt returns error could not access starting directory "C:"" Fortunately C:/ is a valid path for wt

Since we just added a / in this case wouldn't the commandline now expand to C:\/?

rashil2000 avatar Jun 15 '22 04:06 rashil2000

How has this been tested?

Well, here is an example: While in disk volume C, the %V returns C:\, and the command being "wt.exe" -d "C:\", somehow the backslash is treated as a escape character in this situation, then wt returns error could not access starting directory "C:"" Fortunately C:/ is a valid path for wt

Since we just added a / in this case wouldn't the commandline now expand to C:\/?

That's possible... But according to the error report above, I think that may finally be C:/. I guess that by using the variable %V, only path for drive root folder ends with a backslash, which is misprocessed and causes this error. Anyway, both wt.exe -d C:/ and wt.exe -d C:\/ are workable in this case.

AkariiinMKII avatar Jun 15 '22 06:06 AkariiinMKII

And I found "wt.exe" -d "." also works in drive root folder, maybe we should turn to use this?

AkariiinMKII avatar Jun 15 '22 07:06 AkariiinMKII

I think it will be better to see how Windows Terminal installed from the Store does it

Related https://github.com/ScoopInstaller/Versions/pull/557

rashil2000 avatar Jun 15 '22 08:06 rashil2000

Is this related to #9333 ?

jasongodev avatar Mar 17 '23 11:03 jasongodev

Is this related to #9333 ?

yes, that's another solution.

AkariiinMKII avatar Mar 17 '23 11:03 AkariiinMKII

#9333

HUMORCE avatar Apr 11 '23 21:04 HUMORCE