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

Check for broken symbolic links on warehouse load #148

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

creideiki
Copy link
Contributor

When encountering a symbolic link in the warehouse, check that the target exists before following it, and if not abort with an error naming both the source and the target.

Closes #133.

I don't think this merits a version bump or changelog entry in and of itself.

When encountering a symbolic link in the warehouse, check that the
target exists before following it, and if not abort with an error
naming both the source and the target.
@creideiki
Copy link
Contributor Author

If #149 gets merged, throwing a PalletJack::WarehouseError would probably be better than the current Errno::ENOENT.

creideiki added a commit to creideiki/palletjack that referenced this pull request Oct 18, 2017
…-133

* ae76a72 Check for broken symbolic links on warehouse load
creideiki added a commit to creideiki/palletjack that referenced this pull request Oct 18, 2017
…-133

* ae76a72 Check for broken symbolic links on warehouse load
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant