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

Script fails, if the folder pico_setup.sh is downloaded to, has a space in it's name #17

Closed
protik09 opened this issue Mar 15, 2021 · 5 comments
Labels
duplicate This issue or pull request already exists

Comments

@protik09
Copy link

protik09 commented Mar 15, 2021

For example,
I download the script to a folder called "Embedded Projects" and it fails.

pico_setup_error_small

Error: ./pico_setup.sh: line 48: cd: too many arguments

Tried this in WSL (Ubuntu 20.04.2 LTS) and native Linux Mint 20.1 on Cinnamon - it fails in both

I fixed the issues in the script and can send a pull request if needed.

protik09 added a commit to protik09/pico-setup that referenced this issue Mar 15, 2021
@aallan
Copy link

aallan commented Mar 15, 2021

Tried this in WSL (Ubuntu 20.04.2 LTS) and native Linux Mint 20.1 on Cinnamon - it fails in both

The script is designed to run on an up to date installation of Raspberry Pi OS. It is not designed to run on other Linux installations, hence the warning message right at the start "Not running on a Raspberry Pi. Use at your own risk!"

protik09 added a commit to protik09/pico-setup that referenced this issue Mar 16, 2021
Temporary fix to remove unrelated info from pull request for Issue raspberrypi#17
@michaelstoops
Copy link

michaelstoops commented Apr 1, 2021

This is also a problem on Raspberry Pi OS:

mstoops@pod-gw ~/the dir/pico-setup
 % ./pico_setup.sh
...
Creating /home/mstoops/the dir/pico-setup/pico
./pico_setup.sh: line 48: cd: too many arguments
mstoops@pod-gw ~/the dir/pico-setup
 % uname -a
Linux pod-gw 5.10.17-v7+ #1403 SMP Mon Feb 22 11:29:51 GMT 2021 armv7l GNU/Linux
mstoops@pod-gw ~/the dir/pico-setup
 % cat /proc/cpuinfo
...
Hardware	: BCM2835
Revision	: a22082
Serial		: 00000000bcfe828d
Model		: Raspberry Pi 3 Model B Rev 1.2
mstoops@pod-gw ~/the dir/pico-setup
 % cat /etc/os-release
PRETTY_NAME="Raspbian GNU/Linux 10 (buster)"
NAME="Raspbian GNU/Linux"
VERSION_ID="10"
VERSION="10 (buster)"
VERSION_CODENAME=buster
ID=raspbian
ID_LIKE=debian
HOME_URL="http://www.raspbian.org/"
SUPPORT_URL="http://www.raspbian.org/RaspbianForums"
BUG_REPORT_URL="http://www.raspbian.org/RaspbianBugs"

@aallan aallan added the duplicate This issue or pull request already exists label Apr 1, 2021
@aallan
Copy link

aallan commented Apr 1, 2021

Yup, and there is an existing PR to fix this, see #18.

@aallan
Copy link

aallan commented Apr 6, 2021

So #18 has been 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?

@lurch
Copy link
Contributor

lurch commented Apr 8, 2021

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

I'm now following that up with the relevant upstream project: msteveb/jimtcl#199

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This issue or pull request already exists
Projects
None yet
Development

No branches or pull requests

4 participants