topic Re: Shell script problem in Operating System - HP-UX

Shell script problem

Barry Deevey — Thu, 30 Nov 2000 10:37:12 GMT

Hi all,
I'm trying to write a script that checks a directory, moves the files that contain data and deletes the remaining 0 length files - Below is my script that doesn't work properly:

for i in `ll /users/admin/output/ | awk '{print $9}'`
do
if [[ -s "$i" ]]
then
print $i
mv /users/admin/output/$i /users/admin/output/keep/$i
fi
done

rm /users/admin/output/late*

Could anybody point me in the right direction?? Also, could you recommend a good book that I could use to learn from (one that contains good examples etc.) Thank you in advance for any respone, they are very much appreciated.
Best Regards, Barry.

Re: Shell script problem

Frederic Soriano — Thu, 30 Nov 2000 10:52:45 GMT

Hello Barry

Try this:

for i in `ll /users/admin/output/ | awk '{print $9}'`
do
if [ -s "$i" ]
then
print $i
mv /users/admin/output/$i /users/admin/output/keep/$i
fi
done

rm /users/admin/output/late*

Double brackets are not needed here, but be sure that there is a space between the opening bracket and the test flag, and before the closing bracket.

ll may be replace by a 'ls -1', more effective, and avoiding an 'awk' pipe.

As a starting point, you should buy the excellent book from O'Reilly: Unix in a nutshell.

Hope this helps!

Regards.

Re: Shell script problem

Dan Hetzel — Thu, 30 Nov 2000 11:07:49 GMT

Hi Barry,

# remove empty files first
find /users/admin/output -type f -size 0 -exec rm {} \;
# move the remaining files but not the keep directory
cd /users/admin/output
mv !(keep) keep

Best regards,

Dan

Re: Shell script problem

Dan Hetzel — Thu, 30 Nov 2000 11:10:35 GMT

Hi again,

I forgot the book....

Unix Power Tools
Jerry Peek, Tim O'Reilly and Mike Loukides

O'Reilly and Associates

ISBN 0-679-79073-x

Dan

Re: Shell script problem

Open Systems — Thu, 30 Nov 2000 11:15:37 GMT

For efficiency in scripting you could remove the ls altogether (if you don't want to use the find method)

eg:

for i in *
do
if [[ -s ${i} ]]
then
mv ${i} ${KeepDir}
else
rm ${i}
done

Re: Shell script problem

Barry Deevey — Thu, 30 Nov 2000 13:25:48 GMT

Just a quick note, the method described by Frederic did not seem to work, I'll continue to play around with it for my own education.

Thank you all for your postings.

Cheers,
Barry.

Re: Shell script problem

Josef Nordtome — Thu, 30 Nov 2000 17:40:41 GMT

Hello Barry,

The problem with your script is that you are trying to test $i from your current directory. You must put in the full path name in the if statement for it to work.

Using Frederic's example (use single brackets), the if statement should read

if [ -s "/users/admin/output/$i" ]

You can also cd to /users/admin/output before doing the for loop. This would allow you to remove the full path from the if statement. Hope this helps.

Josef