What shell are you using?

There is no reason to export a variable that you are not passing to another shell execution environment.

There is no reason to make a file that does not contain any executable text executable. Consider changing:

Code:

if [[ -f "$PpidFile" ]]
then
	/bin/chmod 755 $PpidFile
	Ret=$?
	if [ 0 -eq $Ret ]
	then	...

to something more like:

Code:

if [ -r "$PpidFile" ]
then	...

There is no need for the tr command in your command substitution. And a:

Code:

while read RelatedEachPid
do	if [ "$RelatedEachPid" = "Ppid" ]
	then	continue
	fi
	...
done < "$PpidFile"

would be much more efficient than:

Code:

for RelatedEachPid in `/bin/grep -v "Ppid" $PpidFile | /usr/bin/tr "\n" " "`
do	...
done

There is no done terminating your for loop.

There is no fi terminating your 1st two if statements.

There is nothing that sets the variables PIDkill and date before they are used.

It looks like you might be trying to count the lines in the file named by the variable PIDkill with the command:

Code:

echo "total of pids killed: wc-l $PIDkill -  $date"

but printing the string wc-l - won't do that. If you had set PIDkill to the pathname of a file, then something more like:

Code:

echo "total of pids killed: $(wc -l $PIDkill) -  $date"

might come closer to doing what you seem to be trying to do.