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

Mark directories when compressing to zip regardless of their contents #230

Merged
merged 1 commit into from
Dec 20, 2021

Conversation

sigmaSd
Copy link
Contributor

@sigmaSd sigmaSd commented Dec 14, 2021

To test this:

  • compress a non empty directory with zip
  • run: ouch l dir.zip -> it outputs the dir name correctly (in blue)
  • compress a non empty directory with ouch
  • run: ouch l dir.zip -> it doesn't output the dir name, because it doesn't identify it as a directory

@sigmaSd sigmaSd changed the title Mark files as dirs when compressing to zip regardless of their contents Mark directories when compressing to zip regardless of their contents Dec 14, 2021
@marcospb19
Copy link
Member

I'm not being able to reproduce this error, but I'm trying only with a single empty folder, how can I see it?

The output here is the same for ouch and zip.

I tried ouch in the latest master branch, as well as the PR branch.

@sigmaSd
Copy link
Contributor Author

sigmaSd commented Dec 19, 2021

The folder has to be not empty

mkdir a; touch a/b;
ouch c a ouch_a.zip; zip -r a.zip a

ouch l a.zip and ouch l a_ouch.zip should show differently

@marcospb19
Copy link
Member

Right, sorry for that one, noticed the fix now.

Merging!

@marcospb19 marcospb19 merged commit 7a358ef into ouch-org:master Dec 20, 2021
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

Successfully merging this pull request may close these issues.

2 participants