openzfsonwindows / openzfs

OpenZFS on Linux and FreeBSD
https://openzfs.github.io/openzfs-docs
Other
402 stars 15 forks source link

Wrong ENV PATH on install #383

Closed Mirkic7 closed 1 month ago

Mirkic7 commented 1 month ago

System information

Type Version/Name
Distribution Name Windows 11 23H2
Distribution Version Pro
Kernel Version OS Build 22631.3593
Architecture x64
OpenZFS Version 2.2.3rc5

Describe the problem you're observing

After installation 'zfs' or 'zpool' commands are not in PATH. Checking ENV variables in System variables, it shows PATH for ZFS is added as: ;C:\Program Files\OpenZFS On Windows; - with semicolons in front and back. Removing them manually fixes PATH issue.

Describe how to reproduce the problem

Fresh install of ZFS on Windows 2.2.3rc5

Include any warning/errors/backtraces from the system logs

Screenshot 2024-05-28 112823

lundman commented 1 month ago

Hmm I just use Inno-setup snippets for it

https://github.com/openzfsonwindows/openzfs/blob/development/contrib/windows/Inno.Setup/environment.iss#L5-L23

sskras commented 1 month ago

Describe the problem you're observing

After installation 'zfs' or 'zpool' commands are not in PATH. Checking ENV variables in System variables, it shows PATH for ZFS is added as: ;C:\Program Files\OpenZFS On Windows; - with semicolons in front and back. Removing them manually fixes PATH issue.

Describe how to reproduce the problem

Fresh install of ZFS on Windows 2.2.3rc5

Can you please open CMD console and run this command:

echo %PATH%

At least zfswin-2.2.3rc4 it works on my laptop just fine:

image

Mirkic7 commented 1 month ago

Hi,

checked PATH as you mentioned, and this is result (reinstalled from scratch). What is 'extra' there is an double quotation mark, which should not be there.

image

lundman commented 1 month ago

Since we add Paths := Paths + ';'+ Path +';' and the " comes before ; that suggests it was there before we added ZFS ?

Mirkic7 commented 1 month ago

Indeed, you are right. I rolled back snapshot, and it was there previously.

False flag, can be closed.

lundman commented 1 month ago

phew! Thanks.