when i am running scripts it is giving error
./dist_auar_pb.sh: line 156: [: /psoft/batch/fsdev/outbound/auar/atl/archive/AUARCPPSF05.120504: binary operator expected

for code
if [ -f $ARC_DIR/${EXTRACT}* ]
then
find $ARC_DIR/${EXTRACT}* -mtime +45 -exec rm {} \;
fi

but clone copy of this script working fine for same coding
y this??
plz tell me solution what should i do with this script why only this script giving error and not clone scripts

presumably the *'s are causing multiple file names to be returned. You can't ask if multiple files exist, nor can you find them. when the script works, there is probably only 0 or 1 file that exists.

Your code doesn't really make sense, you seem to want to say "if there are any files matching this pattern, then go find them and delete them". Why bother checking first? Just delete all the ones that are found. If none are found, none are deleted, so you get the same result, but in much less code.

@chris
Actually folder contain many other files..
So i am dealing with only start A....*

but the question is it is working fine in other places and why giving problem to this only..

as I said, I expect there aren't multiple files in those other locations.

"binary operator expected" means "I've been given more strings to process that the operator expects"

if you script something like "if [ -f file1 file2 file3 file4 ]" then what would you expect to happen? what would that mean??

@chris
yes that true..
i think ir wrks bz that tim having one file in folder and i run script..
and about this script when i run it that time more than one files were there..
thats y it failed...

so do you know how i should check for multiple file??
should i use for??

I've already given you my suggestion. Don't check. just do it. How do you benefit from checking?

please try to write full words... y? bz?

Please use [code][/code] tags around your code and data, to preserve formatting and to improve readability. Please do not use quote tags, colors, or other fancy formatting.


In addition to the above comments, the syntax of the find command is wrong.

You start by giving find one or more starting directories, not files, and various criteria for matching the contents of those directories. Finally you supply an action for find to take when it matches something (-print by default).

One of the actions available in gnu find is "-delete", BTW.

So you really want something more like this:

Since I don't know exactly what your "$EXTRACT" variable contains, I'm leaving it out of the above for the time being. You'll have to figure out how to apply it correctly yourself. Hint: you can give it multiple entries like "-name" at once.

Here are a couple of good links about using find:
http://mywiki.wooledge.org/UsingFind
http://www.grymoire.com/Unix/Find.html


Finally, QUOTE ALL OF YOUR VARIABLE SUBSTITUTIONS. You should never leave the quotes off a parameter expansion unless you explicitly want the resulting string to be word-split by the shell (globbing patterns are also expanded). This is a vitally important concept in scripting, so train yourself to do it correctly now. You can learn about the exceptions later.

http://mywiki.wooledge.org/Arguments
http://mywiki.wooledge.org/WordSplitting
http://mywiki.wooledge.org/Quotes