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

"lein" script does not tolerate paths with spaces. #186

Closed
seh opened this issue Mar 31, 2011 · 2 comments
Closed

"lein" script does not tolerate paths with spaces. #186

seh opened this issue Mar 31, 2011 · 2 comments
Assignees

Comments

@seh
Copy link

seh commented Mar 31, 2011

Per my annotation of line 30 in file bin/lein in commit 42f8f3a, if the base directory of the script contains spaces, the path will be accidentally tokenized as an argument to the find command. Quoting the $LEIN_DIR/lib argument alleviates that problem.

I also noticed that the arguments controlling the depth to which find will search and the apparent error redirection look to be corrupted. At least on Cygwin, I had to edit the aforementioned command at line 30 to make it parse properly:

LEIN_LIBS="$(find -H "$LEIN_DIR/lib" -mindepth 1 -maxdepth 1 -print0 ...
@ghost ghost assigned danlarkin Apr 12, 2011
@seh
Copy link
Author

seh commented Apr 12, 2011

Thank you!

@technomancy
Copy link
Owner

Sure thing. Apparently putting redirections in the middle of lines is technically OK with sh, though of course stylistically it's nutty.

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

3 participants