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

scons install fails if the working directory contains a space #2037

Open
sevu opened this Issue Sep 24, 2017 · 2 comments

Comments

Projects
None yet
3 participants
@sevu
Member

sevu commented Sep 24, 2017

If the wesnoth repo is placed in directory with a space scons install fails at this point:
https://github.com/wesnoth/wesnoth/blob/master/SConstruct#L643

other than in this case there seem to be no problems with quoting

@GregoryLundberg

This comment has been minimized.

Show comment
Hide comment
@GregoryLundberg

GregoryLundberg Sep 24, 2017

Contributor

try Action("chown %s:%s \"%s\"")? Somehow I doubt it'll work but it's worth a shot.

Contributor

GregoryLundberg commented Sep 24, 2017

try Action("chown %s:%s \"%s\"")? Somehow I doubt it'll work but it's worth a shot.

@sevu

This comment has been minimized.

Show comment
Hide comment
@sevu

sevu Sep 24, 2017

Member

Didn't work.

...
Install file: "doc/man/zh_CN/wesnothd.6" as "/tmp/some split directory/pkg/wesnoth-devel/usr/share/man/wesnoth-devel/zh_CN/man6/wesnothd.6"
Install file: "doc/man/zh_TW/wesnothd.6" as "/tmp/some split directory/pkg/wesnoth-devel/usr/share/man/wesnoth-devel/zh_TW/man6/wesnothd.6"
Mkdir("/tmp/some split directory/pkg/wesnoth-devel/run/wesnothd-devel")
Chmod("/tmp/some split directory/pkg/wesnoth-devel/run/wesnothd-devel", 0700)
chown : /tmp/some split directory/pkg/wesnoth-devel/run/wesnothd-devel
chown: Zugriff auf '/tmp/some' nicht möglich: Datei oder Verzeichnis nicht gefunden
chown: Zugriff auf 'split' nicht möglich: Datei oder Verzeichnis nicht gefunden
chown: Zugriff auf 'directory/pkg/wesnoth-devel/run/wesnothd-devel' nicht möglich: Datei oder Verzeichnis nicht gefunden```
Member

sevu commented Sep 24, 2017

Didn't work.

...
Install file: "doc/man/zh_CN/wesnothd.6" as "/tmp/some split directory/pkg/wesnoth-devel/usr/share/man/wesnoth-devel/zh_CN/man6/wesnothd.6"
Install file: "doc/man/zh_TW/wesnothd.6" as "/tmp/some split directory/pkg/wesnoth-devel/usr/share/man/wesnoth-devel/zh_TW/man6/wesnothd.6"
Mkdir("/tmp/some split directory/pkg/wesnoth-devel/run/wesnothd-devel")
Chmod("/tmp/some split directory/pkg/wesnoth-devel/run/wesnothd-devel", 0700)
chown : /tmp/some split directory/pkg/wesnoth-devel/run/wesnothd-devel
chown: Zugriff auf '/tmp/some' nicht möglich: Datei oder Verzeichnis nicht gefunden
chown: Zugriff auf 'split' nicht möglich: Datei oder Verzeichnis nicht gefunden
chown: Zugriff auf 'directory/pkg/wesnoth-devel/run/wesnothd-devel' nicht möglich: Datei oder Verzeichnis nicht gefunden```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment