On Wed, 13.05.15 14:24, Dimitri John Ledkov (dimitri.j.led...@intel.com) wrote:

> On 13 May 2015 at 14:00, Lennart Poettering <lenn...@poettering.net> wrote:
> > On Mon, 11.05.15 16:58, Dimitri John Ledkov (dimitri.j.led...@intel.com) 
> > wrote:
> >
> > Hmm, all the other AM_PATH_PROG variables do not carry a _BIN suffix,
> > should these two? I don't think so.
> >
> > (In general, I am not too big a fan of abbreviating things
> > unnecessarily, unless this is commonly done elsewhere...)
> >
> 
> There is already MOUNT define used elsewhere in the code...
> DEFINE_CAST(MOUNT, Mount);
> 
> Hence the _BIN suffix.

I see... But "_BIN" sounds weird... Please use "_PATH" then, which is
already used for some of the paths configured in Makefile.am

Lennart

-- 
Lennart Poettering, Red Hat
_______________________________________________
systemd-devel mailing list
systemd-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/systemd-devel

Reply via email to