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

Windows Path Destroyed #421

Open
DACN opened this issue Dec 6, 2018 · 8 comments
Open

Windows Path Destroyed #421

DACN opened this issue Dec 6, 2018 · 8 comments

Comments

@DACN
Copy link

DACN commented Dec 6, 2018

I have just installed shoes (shoes-3.3.6-gtk3-32.exe) on windows 10. In doing so, it has just destroyed the Path environment variable, leaving shoes at the only entry. The installation of a tonne of other programs destroyed by installing shoes.

@ccoupe
Copy link

ccoupe commented Dec 7, 2018

Damn! That problem was fixed a few years ago. It must have come back. So Sorry!

FWIW, it's likely to be the NSIS installer was built w/o 'long string' support.

@DACN
Copy link
Author

DACN commented Dec 7, 2018 via email

@ccoupe
Copy link

ccoupe commented Dec 21, 2018

FWIW, I've tested Shoes 3.3.7 (beta) and it cut off a path that was roughly 2700 bytes long. That's with the long string patch (supposedly). How big was your path, @DACN ?

@ccoupe
Copy link

ccoupe commented Dec 21, 2018

Interesting backaground. If I understand it, 2048 characters is the practical limit, and you'd want to be a bit less than that.

ccoupe pushed a commit that referenced this issue Jan 13, 2019
  Still installs Shoe but doesn't kill the PATH
@ccoupe
Copy link

ccoupe commented Jan 13, 2019

Fixed. If the PATH + addition > NSIS_MAX_STRLEN then user will get a warning and the path won't be modified. That works even if the big string patch is not used.

@ccoupe ccoupe closed this as completed Jan 13, 2019
@DACN
Copy link
Author

DACN commented Jan 13, 2019 via email

@Ryokai96
Copy link

Ryokai96 commented Jun 15, 2020

I have just installed (shoes-3.3.7-gtk3-32.exe) on Windows7 and Windows10, it have the same problem, I haven't got warning and my windows path has modified. Fix it, please.

@ccoupe
Copy link

ccoupe commented Jun 15, 2020

@Ryokai96 - Sorry for the difficulty. It must be an off-by-one problem. Be aware the Shoes is no longer being maintained so this problem is unlikely to be fixed.

@ccoupe ccoupe reopened this Jun 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants