Bash script issue (for loop inside a loop)

Mperonen · 08-08-2013, 12:52 AM

I have done the following script (below) to unlock all directories, which it does.

PROBLEM: How could I implement a second for loop inside the first one to loop through the current directory ($file) and to unlock all the content inside that specific directory?

Code:

for file in *
do
  sudo chmod ugo+rwx $file
done

Firerat · 08-08-2013, 01:15 AM

Are you certain you want to do that?

making things rwxrwxrwx is seldom a good idea

For the purpose of illustration

Code:

for file in *
do
  if [ -d $file ]; then
    pushd $file
      sudo chmod ugo+rwx *
    popd
  else
    sudo chmod ugo+rwx $file
  fi
done

BUT that will only chmod files in current dir and 'one level below' dir

if you wanted to chmod ALL files/dir below current ( excluding hidden files/dirs, i.e. those being with "." , that are in the current), then

Code:

chmod ugo+rwx * -R

seriously, it is a bad idea.

why do you want/need to do this?

Mperonen · 08-08-2013, 01:30 AM

Sometimes when using certain tools like file recovery etc the recovered files are automatically locked.
So I use this script only to unlock the files I really need to get rid of.

So yeah I'm sure of it (not unlocking anything important).

Thank you for your answer!

Firerat · 08-08-2013, 02:14 AM

well in that case,

chmod with -R
works the same for chown

but still I would use 755 ( rwxr-xr-x )

Code:

sudo chown <yourusername>:<yourusername> /path/to/recovered -R
sudo chmod 755 /path/to/recovered -R

and really 755 is too much, should be 644 for files and 755 for directories

Code:

sudo find /path/to/recovered -type f -exec chmod 644 {} ';'
sudo find /path/to/recovered -type d -exec chmod 755 {} ';'
sudo chown <yourusername>:<yourusername> /path/to/recovered -R

and if you are writing scripts to do it, you might as well do it safe