- Make slim service stoppable again, since X changed to Xorg - Remove a useless grep call, as using brackets in a grep pattern and piping into "grep -v grep" is redundant - Remove an extra leading tab
Created attachment 215379 [details] Patch file generated with svn diff
@Reporter/maintainer: Is quarterly also affected?
(In reply to Kubilay Kocak from comment #2) Yes, I think so. Besides, we would obviously require a PORTREVISION bump.
(In reply to Kubilay Kocak from comment #2) From now I will always capitalize the first word of the summary ;-) Did you remove the [patch] prefix to increase readability?
(In reply to Samy Mahmoudi from comment #4) Acutally I do not think the grep -v grep is redundant since ps also catches the grep command it self. could you test with pgrep Xorg | xargs ps -p | grep 'auth /var/run/slim.auth' instead?
(In reply to Jesper Schmitz Mouridsen from comment #5) pgrep -f "Xorg.*-auth /var/run/slim.auth" might be better
(In reply to Jesper Schmitz Mouridsen from comment #5) > Actually I do not think the grep -v grep is redundant > since ps also catches the grep command itself. ps does catch and output the grep command, but the following is still redundant: grep '/bin/[X]org .* -auth /var/run/slim.auth' | grep -v grep I think you missed out on the brackets in my point. I did not say the second grep command is useless just because we use grep before. I am saying the second grep command is useless because the first one already filters out the grep command from the output of ps: The shell does not expand quoted strings, so that grep '/bin/[X]org .* -auth /var/run/slim.auth' will become grep /bin/[X]org .* -auth /var/run/slim.auth in the output of ps (no more quote, but brackets still there!). Since the basic regular expression [X] matches the single character X but not the sequence of three characters [X], the first grep will possibly match a line containing /bin/Xorg .* -auth /var/run/slim.auth but certainly not the line containing /bin/[X]org .* -auth /var/run/slim.auth that corresponds to itself. Therefore the subsequent command grep -v grep is useless. > pgrep -f "Xorg.*-auth /var/run/slim.auth" might be better pgrep -f "Xorg.*-auth /var/run/slim.auth" is definitely better than using such a tricky bracket expression ;-)
Created attachment 215803 [details] Patch file generated with svn diff
(In reply to Kubilay Kocak from comment #2) What I meant by 'Yes, I think so' is that I know for sure x11/slim has not recently changed in that regard and that I think x11-servers/xorg-server has neither done so recently. This X-to-Xorg change seems old enough to me to make quarterly affected by this problem as well.
A commit references this bug: Author: jsm Date: Sun Jun 28 14:58:44 UTC 2020 New revision: 540725 URL: https://svnweb.freebsd.org/changeset/ports/540725 Log: x11/slim: Correct getting xpid Makes slim service stoppable again PR: 247098 Submitted by: Samy Mahmoudi <samy.mahmoudi@gmail.com> Changes: head/x11/slim/Makefile head/x11/slim/files/slim.in
(In reply to commit-hook from comment #10) Thanks ;-)