On Wed, 12 Aug 2015 14:17:16 +0200 Julien Cristau <jcris...@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
