Re: [PATCH] udev: sync udev rules directories with systemd units directories

On Sat, Jul 14, 2012 at 12:48 AM, William Hubbs <[email protected]> wrote:
> On Fri, Jul 13, 2012 at 11:38:24PM +0200, Kay Sievers wrote:

>> >                                 TEST_PREFIX UDEVLIBEXECDIR "/rules.d",
>> > +                               TEST_PREFIX "/usr/lib/udev/rules.d",
>> > +#ifdef HAVE_SPLIT_USR
>> > +                               TEST_PREFIX "/lib/udev/rules.d",
>> > +#endif

Now we have 3 instead of one directory here, that really needs to be
sorted out. It's not cool to just add every random thing here. :)

>> I have no problem with that, even when I think it's the wrong way to
>> do these things. It just creates mess, that is much bigger than the
>> split-usr ever was. But we have it in systemd, and there is no real
>> argument for me to make against it.
>
> I'm personally not concerned with /usr/local, but the portion above is
> the more important to me. For split /usr systems, there is a lot of
> software that installs rules in /lib/udev, so I think we need to support
> that directory.

You are aware that this will not work with the installed binaries in
/lib/udev/ referenced from rules and whatever? The code is really not
made to have this I-read-stuff-from-all-possible locations.

It's a mess you are creating here, by not just merging /lib and
/usr/lib right away.

> Should I resubmit the patch?

Nah, we need to decide what to do first, I'll let you know.

Kay
--
To unsubscribe from this list: send the line "unsubscribe linux-hotplug" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html