Ventoy icon indicating copy to clipboard operation
Ventoy copied to clipboard

[issue]: VentoyPlugson.exe does not accept lowercase drive letter

Open steve6375 opened this issue 1 year ago • 4 comments

Official FAQ

  • [X] I have checked the official FAQ.

Ventoy Version

1.0.85

What about latest release

Yes. I have tried the latest release, but the bug still exist.

Try alternative boot mode

Yes. I have tried them, but the bug still exist.

BIOS Mode

Both

Partition Style

MBR

Disk Capacity

128gb

Disk Manufacturer

sandisk

Image file checksum (if applicable)

None

Image file download link (if applicable)

No response

What happened?

Two issues:

font file is "\ventoy\themes\ascii.pf2" USB drive is formatted as NTFS.

  1. When specifying path for font, k:\ventoy\themes\ascii.pf2 is not accepted - i.e. lowercase drive letter is not accepted as valid path.

  2. However, K:\ventoy\themes\ascii.PF2 and .pF2 is accepted - i.e. case of extension is wrong but this is accepted.

image

GUI does not mention drive letter is case sensitive and does not warn user that they should check path and filename for case sensitivity.

steve6375 avatar Dec 20 '22 09:12 steve6375

Q1: I will fix it soon.

Q2: In Windows, by default file path is case insensitive, so you can input file path in lower/upper case. But for NTFS Ventoy will automatically use the right case in ventoy.json. So you can preview the json content or see the ventoy.json file, the path in the json is right.

ventoy avatar Dec 20 '22 10:12 ventoy

Q2 - yes you are right - but I can add multiple entries all with different cases - which is a bit confusing "/ventoy/themes/ascii.pf2", "/ventoy/themes/ascii.pf2", "/ventoy/themes/ascii.pf2"

steve6375 avatar Dec 20 '22 10:12 steve6375

P.S. If I edit the json manually in Notepad while using Ventoy_plugson - e.g. add "ventoy_color" and "ventoy_left" lines and save the file in the editor. If I then make a change in Ventoy_plugson, my changes are lost. I must click on Stop in Windows .exe pop-up and then 'Start' to reload the json file. Could there be added a 'Reload ventoy.json' text link in the web app (or maybe detect if file has changed and automatically reload it into web GUI?).

or add all global entries such as ventoy_color, etc. into web app?

steve6375 avatar Dec 20 '22 11:12 steve6375

Q1 has been fixed. And also I add check for the duplicate path in different case.

Reload will make the implement a little complex and most of the users only use VentoyPlugson and don't manually edit json file at the same time. So I will not add reload feature now. Just close the exe before you manually edit the json file.

For ventoy_left/ventoy_top/ventoy_color , I hope users leave this information, so I don't add them in the Web page :) If you want, just edit the json file manually. If these options are already in the json file before you run the exe, then Plugson will load them and will not change them.

ventoy avatar Dec 20 '22 12:12 ventoy

Fixed in 1.0.86

ventoy avatar Dec 25 '22 02:12 ventoy