[PATCH xinit] startx: Make the bugzilla reference in a comment survive cpp processing
Alan Coopersmith
alan.coopersmith at oracle.com
Wed Aug 27 09:15:21 PDT 2014
On 08/26/14 02:53 AM, Hans de Goede wrote:
> https://bugs.freedesktop.org/show_bug.cgi?id=83019
>
> Signed-off-by: Hans de Goede <hdegoede at redhat.com>
> ---
> startx.cpp | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/startx.cpp b/startx.cpp
> index 5dafce7..1c6fce0 100644
> --- a/startx.cpp
> +++ b/startx.cpp
> @@ -191,7 +191,7 @@ if [ x"$server" = x ]; then
> #ifdef __linux__
> XCOMM When starting the defaultserver start X on the current tty to avoid
> XCOMM the startx session being seen as inactive:
> - XCOMM https://bugzilla.redhat.com/show_bug.cgi?id=806491
> + XCOMM "https://bugzilla.redhat.com/show_bug.cgi?id=806491"
> tty=$(tty)
> if expr match "$tty" '^/dev/tty[0-9]\+$' > /dev/null; then
> tty_num=$(echo "$tty" | grep -oE '[0-9]+$')
>
I was thinking \ when I saw the bug, but quotes make it easier to read &
copy/paste out of the source, so that's a nice solution.
Reviewed-by: Alan Coopersmith <alan.coopersmith at oracle.com>
--
-Alan Coopersmith- alan.coopersmith at oracle.com
Oracle Solaris Engineering - http://blogs.oracle.com/alanc
More information about the xorg-devel
mailing list