raspberrypi / pico-setup

92 stars 42 forks source link

Fix spaces in parent folder name #18

Open protik09 opened 3 years ago

protik09 commented 3 years ago

This commit fixes Issue #17

aallan commented 3 years ago

Not needed on Raspberry Pi OS, but might be worthwhile merging in any case?

JamesH65 commented 3 years ago

Why not needed on PiOS? If there is space in the installation folder name, I think pico-setup would fail?

aallan commented 3 years ago

Why not needed on PiOS? If there is space in the installation folder name, I think pico-setup would fail?

Doh! Fair point. I was totally in the mindset of "well it works on my machine!" Blame it on Monday and not enough coffee? 🤦‍♂️

aallan commented 3 years ago

Superseded by #20. But note that one of openocd's submodules doesn't handle spaces and there doesn't seem to be a way around that?