On 11/21/19 1:48 PM, Todd Zullinger wrote:
Samuel Sieb wrote:
On 11/21/19 1:16 PM, home user wrote:
if ! [[ "pgrep xeyes > /dev/null" ]] ; then
This is still the problem. Remove those quotes. And you could remove the brackets as well, they aren't necessary.
Indeed. And while pidof would have probably been a better solution for the initial problem we thought we were trying to solve, I think knowing that you only want to skip user1 from starting new xeyes if it is already running for user1 that pgrep will be better suited. With pgrep you can use the -u/--euid or -U/--uid option to limit the match to xeyes processes which are running for the proper user.
I don't think that would work in this case. The problem is that he's switching users, so from one user account he would need to check for the processes under the other user, not the current one. Assuming that he's only going to have one user logged in to the desktop at a time, this is the best option. Although "pidof" should work equally well.