-
Notifications
You must be signed in to change notification settings - Fork 49
Fix unjustified warning about casing for directory entries #155
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
Fix unjustified warning about casing for directory entries #155
Conversation
src/test/java/org/codehaus/plexus/archiver/AbstractUnArchiverTest.java
Outdated
Show resolved
Hide resolved
Thanks, @michael-o for the review. I don't have the rights to merge this PR, I'd be grateful if someone who does have the rights can merge it for me. |
@michael-o I see a lot of Maven components already updating Plexus Archiver, with this little issue still present. Shouldn't we try to merge this in, too, given the false warnings that'll occur otherwise? |
@mthmulders would you please squash the commits into one and I'll merge it. |
9feba71
to
a503307
Compare
Squashed & force-pushed. Thanks in advance :-) |
The build is failing after the merge. I'll check it later today |
My guess is that's unrelated to the code change, but feel free to investigate further and let me know if you draw a different conclusion. |
Correct, is an ephemeral network issue. Restart or check your internet connection. |
@mthmulders I've released a new version with this fix. Thanks for contributing. |
Thank you for considering this contribution and for releasing a new version so quickly. |
Follow-up for #128, where we omitted the case that an archive entry could be a directory. In that case, its name will end with
/
, but the canonical target path will not have the/
suffix. This would be seen as a difference in casing, but that is not true.