Hi there --

I am trying to debug a script that I wrote to monitor disk usage on one of our servers with an automatic notification to those who exceed the limit. Here is the text of the script:

#!/bin/bash

# find disk hogs

users='ls -1 /home'
limit=100000

for user in $users
do
diskuse='du -hcs /home/$user | awk '{ print $1 }' -'
if [ $diskuse -gt $limit ]
then
/bin/mail $user <<!

Dear $user,

It is expected that users on this system keep their disk usage below $limit. You are currently using $diskuse. Please delete any unnecessary files and directories from your home directory.

I can help you archive old files and directories if you wish. Please let me know if there are any that can be archived to tape, and I will be happy to do the necessary backups. If you have any questions, please feel free to contact me at 617-724-6949.

Thank-you for your time and patience on this matter.

!
fi
done

When I tried to run the script, the following errors repeatedly appeared onscreen:

line 10: print: command not found
line 11: [: -gt: unary operator expected

The operating system is Fedora Core 5. Does anyone have any insight to this? Thanks.

Try
diskuse=$(du -s /home/$user | awk '{ print $1 }')

Edit: dropped the -h because of the comparison you do afterwards

Nnnnnever mind. uselpa beat me to it!

I made the change that you suggested, and the error messages I am now getting are the following:

du: cannot access '/home/ls' : no such file or directory
line 11: [: -gt:unary operator expected
du: cannot access '/home/-1' : no such file or directory
line 11: [: -gt:unary operator expected
du: cannot access '/home/home' : no such file or directory
line 11: [: -gt:unary operator expected

users=$(ls -1 /home)

That seems to have fixed it. One question: if I want the limit mentioned in the e-mail
that is sent out to read megabytes, can I simply change the limit= line from:

limit=80000

to

limit=80MB

Thanks.

no, because the comparison wouldn't work
if you know it's MB then divide the amount by 1024*1024 or something along those lines