Question

I work in a research group and we use the PBS queuing system. I’m no PBS master, but I wanted to script a search for if a job was running. To do this I first grab a string of all the jobs by using the results of a qstat call as my argument to qstat -f and then taking the detailed list of all jobs and searching it for the submitted file path. The current kludge stands as follows

dump=`qstat -f `qstat``
if grep -q \
          "/${compounds[$i]}/D0_${j}_z_$((k*30))/scripts/jobscript_minim" \
      <<<$dump; then
      echo "Minimize is running!"
fi

Suggestions for improvement?

Also, I’ve been told that $() is cleaner than “. But when I try:

dump="$(qstat -f "$(qstat)")"

…my program fails. Why is this? Am I misunderstanding how to nest shell calls with $()?? Or is it something to do with how I’m passing the list of queue jobs from qstat to qstat -f? Should I be using awk or something to grab the jobs from the qstat command and then somehow pass them as args to qstat -f?

Also should I be using recursive grep? Some people tell me its “saner” but I’m not sure what that means. Is it more portable? Is it faster? Does it need less trips to the therapist?

What is the reason you should use it?

Answer 1

Alright… managed to come up with a clean solution…

search_dir="${compounds[${i}]}/D0_${j}_z_$[30*k]"
if [ ! -z "$(qstat -f $(qstat | grep -F jmick | awk '{print $1}')|\
      grep -F "$search_dir"|head -n 1)" ]
      then

…since the directory I’m searching for is kind of long I assign it to a variable. I run the inner command substitution to get only the jobs with my user name, then run the outer command substitution to print full details on those jobs and then grep through those details for my directory. In case it finds it early I included a head to try to short circuit the command.

The question of what’s the point of recursive grep, though, still stands.