CODEWITHSUNDEEP
bashshelldirectoryunzip
new_programmer_22
new_programmer_22

Reputation: 475

Loop through and unzip directories and then unzip items in subdirectories

I have a folder designed in the following way:

-parentDirectory
---folder1.zip
----item1
-----item1.zip
-----item2.zip
-----item3.zip
---folder2.zip
----item1
-----item1.zip
-----item2.zip
-----item3.zip
---folder3.zip
----item1
-----item1.zip
-----item2.zip
-----item3.zip

I would like to write a bash script that will loop through and unzip the folders and then go into each subdirectory of those folders and unzip the files and name those files a certain way.

I have tried the following

cd parentDirectory
find ./ -name \*.zip -exec unzip {} \;

count=1

for fname in *
do
    unzip
    mv $fname $attempt{count}.cpp
    count=$(($count + 1))
done

I thought the first two lines would go into the parentDirectory folder and unzip all zips in that folder and then the for loop would handle the unzipping and renaming. But instead, it unzipped everything it could and placed it in the parentDirectory. I would like to maintain the same directory structure I have.

Any help would be appreciated

Upvotes: 1

Views: 2980

Answers (1)

Kenny Ostrom
Kenny Ostrom

Reputation: 5871

excerpt from man unzip

[-d exdir] An optional directory to which to extract files. By default, all files and subdirectories are recreated in the current directory; the -d option allows extraction in an arbitrary directory (always assuming one has permission to write to the directory).

It's doing exactly what you told it, and what would happen if you had done the same on the command line. Just tell it where to extract, since you want it to extract there.

see Ubuntu bash script: how to split path by last slash? for an example of splitting the path out of fname.

putting it all together, your command executed in the parentDirectory is

find ./ -name \*.zip -exec unzip {} \;

But you want unzip to extract to the directory where it found the file. I was going to just use backticks on dirname {} but I can't get it to work right, as it either executes on the "{}" literal before find, or never executes.

The easiest workaround was to write my own script for unzip which does it in place.

> cat unzip_in_place
unzip $1 -d `dirname $1`

> find . -name "*.zip" -exec ./unzip_in_place {} \;

You could probably alias unzip to do that automatically, but that is unwise in case you ever use other tools that expect unzip to work as documented.

Upvotes: 2

Related Questions