Parameter list is too long.

[Rafael Mendonça Braga]

Hello There...
I have a Script that removes logs with "x" days in a directory of my HPUX 11i server.
The logs are generated following a standard:
Dxx_LOG_NAME.log, where xx is the number of days we have to retain the log.
The scripts is this:

DAY=$(ls -1 /interface/web/D* |cut -f2 -dD |cut -f1 -d_)

NEW=00
for EACHONE in $DAY
do
if [ "${NEW}" -ne "${EACHONE}" ]; then
NEW=$EACHONE
EACHONE=`expr $EACHONE \* 1`
EACHONE=`expr $EACHONE - 1`
find /interface/web/ -name D$NEW*.* -mtime +$EACHONE -print -exec rm {} \;
fi
done

----------------------------------------------------------

Sometimes I'm receiving this message:

sh: /usr/bin/ls: The parameter list is too long.

Do you Know How can I avoid it?

Thanks,

Rafael

[RAC_1]

You are hitting MAX_ARGS limit. Pipe the output and then do args on it to do what you want.

[Steven E. Protter]

ls -1 > file

while read -r filename
do
rm -f $filename
# or any other operation you want to accomplish.
done < file


SEP

[James R. Ferguson]

Hi:

You are causing the shell to collect create a list that is too long. Change the way you drive your script to leverage the 'find' of the directory, like:

...
cd /interface/web
find . -type f |
while read FILE
do
...
done

Regards!

...JRF...

[Roy Colica · Accepted solution]

Rafael,
use find instead of ls to solve the issue.
Re
Roy

[Doug O'Leary]

Hey;

Although the other posts would work, RAC mentioned the correct answer:

DAY=$(ls -1 /interface/web/D* |cut -f2 -dD |cut -f1 -d_)

NEW=00
for EACHONE in $DAY
do
if [ "${NEW}" -ne "${EACHONE}" ]; then
NEW=$EACHONE
EACHONE=`expr $EACHONE \* 1`
EACHONE=`expr $EACHONE - 1`
find /interface/web/ -name D$NEW*.* -mtime +$EACHONE -print | xargs -i rm {}
fi
done

The trick is the xargs pipe from the find command. Minor tweak.

HTH;

Doug

[Doug O'Leary]

Oops; my mistake. Misread where the error was coming from...

I'd still keep the xargs command in your loop; however, you can avoid the potential buffer overflow by:


NEW=00
for EACHONE in $(find /interface/web -name D\* -exec ls -dl {} \; | cut -f2 -dD | cut -f1 -d_)
do
if [ "${NEW}" -ne "${EACHONE}" ]; then
NEW=$EACHONE
EACHONE=`expr $EACHONE \* 1`
EACHONE=`expr $EACHONE - 1`
find /interface/web/ -name D$NEW*.* -mtime +$EACHONE -print | xargs -i rm {}
fi
done

Sorry for the confusion...

Doug

[Rafael Mendonça Braga]

Thanks Guys!!!

It's Working!