chdir("/")||die "Error:$!\n";
opendir(ROOT_DIR,"/")|| die "Error:$!\n";

search_dir("/","/");

sub search_dir{
     my ($dir,$fullpath) = @_;
foreach $dir_cont (sort readdir($dir)){
  next if $dir_cont =~s/^\.\.?$/;
  if (-f $dir_cont){
    log_sbit($full_path . $dir_cont) if (-u $dir_cont || -g $dir_cont)
+;
  }
  elsif (-d $dir_cont && opendir(SUB_DIR,"$dir_cont"){
    my $new_full_path = $full_path . $dir_cont;
    search_dir($dir_cont,$new_full_path);
  }
}
[download]

This code is untested. I just added the $full_path so you could keep track of the full path and I put it all into a sub. So far you have the right idea. You just need to put your logic in a routine you can call over and over again. Once you get it right, check out File::Find.

next if $dir_cont =~s/^\.\.?$/;
[download]

you might want to change that line to:

next if $dir_cont=~/^\.\.?$/;
[download]

because I really don't think you want substitution. basically delete the 's' also

my($dir,$fullpath) = @_;
[download]

should be

my($dir,$full_path) = @_;
[download]

opendir(ROOT_DIR,"/")|| die "Error:$!\n";
my @todo = grep {$_ ne '..'} readdir(ROOT_DIR);
closedir(ROOT_DIR);
while (my $what = shift @todo) {
  if (-f $what){
    log_sbit($what) if (-u $what || -g $what);
  }
  elsif (-d $what && opendir(SUB_DIR, $what){
    push @todo, map {"$what/$_"} grep {$_ ne '..'} readdir(SUB_DIR);
    closedir(SUB_DIR);
  }
}
[download]

And I can't resist: for more information on recursion, see Recursion.

You know, I must have followed that Link for More Info at least twenty times and i never saw anything new...

Ah, but each time you saw it, you had the knowledge gained from all the previous times and hence an ever deeper understanding.

  p

OK I like to know how things work so although you can and probably should do this with a module here is how you do it.

#!/usr/bin/perl -w

use strict;

my $root  = 'c:/cluster1/';
my @dirs  = ($root);
my @files;

for my $path (@dirs){
    opendir ( DIR, $path ) or next; # skip dirs we can't read
    while (my $file = readdir DIR) {
        # skip the dot files
        next if $file eq '.' or $file eq '..';
        # skip symbolic links to avoid infinite loops 
        next if -l $path.$file;
        if ( -d $path.$file ) {
            # add the dirs to our dir list (full path)
            push @dirs, $path.$file.'/';
        } 
        else {
            # add the files to file list       
            push @files, $path.$file;      
        }     
    }
    closedir DIR;
}

print "Directory list\n\n";
print "$_\n" for sort @dirs;

print "\n\nFile List\n\n";
print "$_\n" for sort @files;
[download]

How we do it is simple. First we define our root dir and our delimiter and push this into the @dirs array. We then iterate over the @dirs array. Although @dirs initially only contains the root dir we find all its subdirs and push them onto the end of this. Thus after the first iteration @dirs contains all the subdirs of our root dir which we then search for subdirs and so on ad infinitum until we have no more subdirs. Testing for the . and .. dirs is important as these are the current and one level up dirs. Using eq is better than a regex as you can fool a regex using a newline in a dirname (which is valid). We do not want to follow symbolic links otherwise we may end up in an infinite loop. This is what the guys allude to above.

We then test each file to see if it is a direcory or a file and push it into the appropriate array. By the fime we have iterated over @dirs we have all the directories in @dirs and all the files in @files. These are the fullly specified paths. You can then do whatever you like with them.

Hope this helps.

cheers

tachyon

s&&rsenoyhcatreve&&&s&n.+t&"$'$`$\"$\&"&ee&&y&srve&&d&&print

Fitting very nicely here: Dominus' talk about The Identity Function, especially the slides starting here.

-- Hofmator