search

pal1000 / mesa-dist-win

Pre-built Mesa3D drivers for Windows
MIT License
860 stars 77 forks source link

Improve perappdeploy.cmd #186

Closed clairem-sl closed 2 months ago

clairem-sl commented 2 months ago

perappdeploy.cmd when run through Windows Terminal sometimes doesn't detect Ctrl-C reliably, this makes the batch file continue on after symlink deletion. Making it a bit sketchy when all I want is to just delete the symlinks.

The changes I propose implements a pause before going into the deployment phase, and will continue only if the user answers y.

In addition, I also added empty lines around the last question to make it stand out more.

pal1000 commented 2 months ago

Just pressing enter without answering to all questions would have terminated the script more reliably than CTRL+C. But this is good improvement. Not everyone would have figured that out.