Csdrhrt

I'm trying to make a script to either quit compton if it's running or start it if it's not running. I've read from man that it should exit 1 if process is found, so I've tried to make a script that uses that... However this just doesn't work, It starts if it's closed but doesn't close it. what am I doing wrong ??

#!/bin/bash





status=$(pgrep compton 2>&1)



if [[ $status == 1 ]];

    then

        killall compton

    else

        exec compton -b

fi



echo $status

You are getting the pgrep output in your status variable. It's just not the output that you expect it to be.

pgrep outputs the process IDs (PIDs) of the processes matching the pattern that you give it. If there is a process whose name matches compton, then $status would be the PID of that process, or of those processes. pgrep also returns an exit status, but an exit status is not captured by a command substitution as a string.

In your test, you compare $status against 1. It is unlikely that compton has PID 1.

If you want to kill any compton process if they exist, and start compton -b if no compton process exists, you may do that with

#!/bin/sh



if ! pkill compton; then

    exec compton -b

fi

This uses the exit status of pkill. The pkill tool works in an equivalent way to pgrep (they are usually distributed and installed as a pair) but instead of outputting PIDs of matching processes like pgrep would do, pkill sends the TERM signal (by default) to the matching processes.

The if keyword uses the exit status of the command that you use with it.

The ! inverts the sense of the test so that

• If pkill compton succeeds, it means that there was one or several compton processes that have now been killed, or at least signalled, and exec compton -b will not be executed.




• If pkill compton fails (no process matched the name, or there was some internal error in pkill), the body of the if statement would call your exec compton -b, which would replace the shell process with that of compton -b.

You should control exit status of pgrep process which will be in $? variable. Or check if $status variable where you're storing the output of pgrep is f.e. non zero-length string. The script in the question checks whether string in variable status is "1"

so

#!/bin/bash

pgrep compton >/dev/null



if [[ $? -eq 0 ]]

then

    killall compton

else

    exec compton -b

fi

or

#!/bin/bash

status=$(pgrep compton 2>&1)



if [[ -n "$status" ]]

then

    killall compton

else

    exec compton -b

fi