Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Fix li3 script so that it handles directories with spaces in them. #438

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants
Contributor

phuff commented Apr 25, 2012

I found that if I tried to run li3 test from a directory with spaces in it that the shell script died. So I added some quotes in appropriate places and it works now. I was able to reproduce on Mac OS X and on ubuntu, so I assume it's not just me... :) I don't know if there are any tests for the shell script, so if there are let me know and I'll submit some tests, too.

Member

daschl commented Apr 26, 2012

@phuff thanks for the enhancement, but can you reopen the pull reuqest against the dev branch please? And also, why did you change the year back to 2011?

Contributor

phuff commented Apr 26, 2012

Ah crap. Sorry, I made the changes against a slightly older checkout. I'll close this one and make it against the dev branch with a better commit. Sorry. n00b-ing it up around here :)

@phuff phuff closed this Apr 26, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment