#759518 x11-common: Print the real name of the executable instead of x-session-manager

Package:
x11-common
Source:
xorg
Submitter:
Laurent Bigonville
Date:
2015-08-21 10:18:08 UTC
Severity:
wishlist
#759518#5
Date:
2014-08-27 23:13:23 UTC
From:
To:
Hi,

Currently, when starting a session, x-session-manager (the alternative
symlink) is shown in the process list instead of the real name of the
executable.

I guess the symlink should be followed.

Cheers,

Laurent Bigonville
#759518#10
Date:
2014-09-22 22:21:54 UTC
From:
To:
affects 759518 + apport
thanks

Hi,

Apparently this could actually affects apport (/usr/bin/apport-bug)
that actually try to determine the type of session the user is running
by grepping the process list.

Cheers,

Laurent Bigonville
#759518#17
Date:
2014-09-23 19:25:53 UTC
From:
To:
tag 759518 + patch
thanks

Hi,

Is a patch like this acceptable?

Cheers,

Laurent Bigonville
#759518#24
Date:
2014-09-23 19:47:02 UTC
From:
To:
Le Tue, 23 Sep 2014 21:25:53 +0200,
Laurent Bigonville <bigon@debian.org> a écrit :

Bahh wrong copy/paste I guess we should use
/usr/bin instead of /etc/alternatives/ I guess
#759518#29
Date:
2015-08-12 11:54:19 UTC
From:
To:
Hi,

Yet another version of this patch.

I guess it makes more sense to use the full path of the alternative if
we are testing that it exits just the line before.

Cheers,

Laurent Bigonville
#759518#34
Date:
2015-08-12 12:17:16 UTC
From:
To:
FWIW I'm not convinced that this is a bug, nor something we need to
change.

Cheers,
Julien
#759518#39
Date:
2015-08-21 10:15:36 UTC
From:
To:
On Wed, 12 Aug 2015 14:17:16 +0200 Julien Cristau <jcristau@debian.org> wrote:

Testing for the existence of an executable with the full path and then
relying on the PATH looks like a bug to me.

Printing the name of the executable might be indeed seen as something
cosmetic, that's true. I don't think that adding the call to realpath
is making the code more difficult to read though.

Cheers,

Laurent Bigonville