Bug#759518: x11-common: Print the real name of the executable instead of x-session-manager
On Wed, 12 Aug 2015 14:17:16 +0200 Julien Cristau <jcristau@debian.org>
wrote:
> On Wed, Aug 12, 2015 at 13:54:19 +0200, Laurent Bigonville wrote:
>
> > Package: x11-common
> > Followup-For: Bug #759518
> >
> > 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.
> >
> FWIW I'm not convinced that this is a bug, nor something we need to
> change.
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
Reply to: