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

setup-environment can't handle paths with spaces #13

Closed
koenkooi opened this issue May 22, 2015 · 3 comments
Closed

setup-environment can't handle paths with spaces #13

koenkooi opened this issue May 22, 2015 · 3 comments

Comments

@koenkooi
Copy link
Contributor

From my jenkins logs:

+ MACHINE=beast
+ . ./setup-environment
++ . .repo/manifests/setup-environment-internal
++++ whoami
+++ '[' jenkins = root ']'
+++ OEROOT='/var/lib/jenkins/.jenkins/jobs/Angstrom v2015.06 Angstrom-image builds - manifest/workspace/machine/beast'
+++ cd /var/lib/jenkins/.jenkins/jobs/Angstrom v2015.06 Angstrom-image builds - manifest/workspace/machine/beast
.repo/manifests/setup-environment-internal: line 28: cd: /var/lib/jenkins/.jenkins/jobs/Angstrom: No such file or directory
Build step 'Execute shell' marked build as failure
Collecting metadata...
Metadata collection done.

Looks like it needs to either always use straight quotes or escape it.

kraj added a commit that referenced this issue May 22, 2015
So we can accomodates directories with non alphanumeric characters in
its name

Fixed Issue #13

Change-Id: Id335b60a96f553ec99227d50c2fb0f48999ff61f
Signed-off-by: Khem Raj <raj.khem@gmail.com>
kraj added a commit that referenced this issue May 22, 2015
So we can accomodates directories with non alphanumeric characters in
its name

Fixed Issue #13

Change-Id: Id335b60a96f553ec99227d50c2fb0f48999ff61f
Signed-off-by: Khem Raj <raj.khem@gmail.com>
@koenkooi
Copy link
Contributor Author

It gets a bit further and fails here:

+++ mkdir -p conf
++++ grep '^MACHINE' /var/lib/jenkins/.jenkins/jobs/Angstrom v2015.06 Angstrom-image builds - manifest/workspace/machine/beast/conf/auto.conf
++++ sed -e 's%^MACHINE ?= %%'
++++ sed -e 's/^"//' -e 's/"$//'
grep: /var/lib/jenkins/.jenkins/jobs/Angstrom: No such file or directory
grep: v2015.06: No such file or directory
grep: Angstrom-image: No such file or directory
grep: builds: No such file or directory
grep: manifest/workspace/machine/beast/conf/auto.conf: No such file or directory
+++ oldmach=
+++ '[' -e conf/checksum -a beast = '' ']'
+++ sha512sum /var/lib/jenkins/.jenkins/jobs/Angstrom v2015.06 Angstrom-image builds - manifest/workspace/machine/beast/.repo/manifests/setup-environment-internal
sha512sum: /var/lib/jenkins/.jenkins/jobs/Angstrom: No such file or directory
sha512sum: v2015.06: No such file or directory
sha512sum: Angstrom-image: No such file or directory
sha512sum: builds: No such file or directory
sha512sum: manifest/workspace/machine/beast/.repo/manifests/setup-environment-internal: No such file or directory

@kraj
Copy link
Member

kraj commented May 23, 2015

thinking a bit more, I think spaces in dirname wont work with OE in general because the bblayers are split across spaces too. So it will need changes in bitbake to accommodate directories with spaces.

I think you should convert space to underscores

kraj added a commit that referenced this issue May 23, 2015
Fixes part of Issue #13

Change-Id: Ib8b1f9ed5f730d20c62ab0769d1c548c06814084
Signed-off-by: Khem Raj <raj.khem@gmail.com>
kraj added a commit that referenced this issue May 23, 2015
Fixes part of Issue #13

Change-Id: Ib8b1f9ed5f730d20c62ab0769d1c548c06814084
Signed-off-by: Khem Raj <raj.khem@gmail.com>
@koenkooi
Copy link
Contributor Author

You're right, bitbake/OE can't handle spaces in paths either, I've set the custom workspace var to '/build/jenkins/angstrom-v2015.06' in jenkins for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants