Interlisp / medley

The main repo for the Medley Interlisp project. Wiki, Issues are here. Other repositories include maiko (the VM implementation) and Interlisp.github.io (web site sources)
https://Interlisp.org
MIT License
376 stars 19 forks source link

Windows installer doesn't escape spaces in file paths. #1868

Open kotenok2000 opened 3 weeks ago

kotenok2000 commented 3 weeks ago

Describe the bug When i tried to install medley on Windows i have set install path to E:\Program files\interlisp It installed cygwin to E:\Program To Reproduce Steps to reproduce the behavior:

  1. Run https://github.com/Interlisp/medley/releases/download/medley-241022-5fef8528/medley-full-cygwin-x86_64-241022-5fef8528_240513-4becc6ad.exe
  2. Use install path cotaining spaces.
  3. It installed cygwin to E:\Program and couldn't find bash.

Expected behavior Interlisp handles file paths containing spaces without errors. Screenshots If applicable, add screenshots to help explain your problem.

Context (please complete the following information):

Additional context Add any other context about the problem here.

masinter commented 3 weeks ago

probably should go through all the scripts and make sure parameters are quoted

fghalasz commented 6 days ago

Fixed in Pull Request #1880.