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

Use find instead of ls -d to avoid -bash: /bin/ls: Argument list too long #24

Closed
drags opened this issue Aug 18, 2015 · 2 comments
Closed

Comments

@drags
Copy link

drags commented Aug 18, 2015

First off, thanks for this handy script!

I ran into an issue because my build server had ~85k orphaned volumes (image building server from late 2014 :) ) where the ls -d ${targetdir}/* command was failing.

I was able to replace it with find ${targetdir} -maxdepth 1 -type d and continue on my merry way, however I don't have the time or criteria to test and ensure that works in all cases, so I figured an issue was better than a PR.

Thanks again and hope this helps!

@almonteb
Copy link
Contributor

Hey @drags, I ended up having the same issue and sent over pull req #23. Updated to include maxdepth.

@drags
Copy link
Author

drags commented Aug 19, 2015

@almonteb awesome! I should have checked PRs first. Thank you for opening #23

@drags drags closed this as completed Aug 19, 2015
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