Re: The Next Generation

[Roman Kagan <rkagan@mail.ru>]

On Mon, Feb 28, 2005 at 09:37:29PM +0100, Kay Sievers wrote:
> On Mon, 2005-02-28 at 22:11 +0300, Roman Kagan wrote:
> >HOTPLUG="sh -c 'for f in /etc/dev.d/default/*.dev; do [ -x \$f ] && \$f; done'"
> 
> Sure, something like that should work too. It was easy to do this inside
> of udev, don't know what's better here.

My point was that you wanted udev to be as ligthweight as possible on
one hand, but kept adding functionality, which is not absolutely
necessary, on the other.

> >Another suggestion is of a syntactic sugar kind: would it be too hard to
> >implement key substitution in actions with a generic syntax, rather than
> >adding %-conversion specifier for each possible substitution, e.g.
> >
> >HOTPLUG="foo $SUBSYSTEM $SYSFS{bar} $ENV{baz}"
> 
> No, that is not really hard. I thought about something like this too, if
> we are running out of nice characters. :)

The problem with "nice characters" is that each one need additional
doctumentation, too.

> It is nicer to read, yes. But renaming some of the keys would be nice
> too, like:
>   ADD_SYMLINK, ADD_HOTPLUG, MATCH_ACTION, SET_GROUP, SET_MODE, SET_NAME
> 
> so the key can tell you what it does in reality. Today you need to know
> that KERNEL= is a match instruction, SYMLINK= will add something to a
> list and GROUP= will override any value.

I think that's the distinction between keys to be matched against
(SUBSYSTEM, BUS, ID, etc.), and the actions to be taken (NAME, GROUP,
MODE, HOTPLUG), which was missing from the very beginning.  OTOH I'm
with Marco here, that making it too verbose doesn't make a lot of sense.
At most I would add a separator between keys and actions.

Cheers,
  Roman.


-------------------------------------------------------
SF email is sponsored by - The IT Product Guide
Read honest & candid reviews on hundreds of IT Products from real users.
Discover which products truly live up to the hype. Start reading now.
http://ads.osdn.com/?ad_ide95&alloc_id\x14396&op=click
_______________________________________________
Linux-hotplug-devel mailing list  http://linux-hotplug.sourceforge.net
Linux-hotplug-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-hotplug-devel