From mboxrd@z Thu Jan 1 00:00:00 1970 From: Roman Kagan Date: Mon, 28 Feb 2005 21:25:49 +0000 Subject: Re: The Next Generation Message-Id: <20050228212549.GJ2964@katya> List-Id: References: <20050217190941.GA1561@vrfy.org> In-Reply-To: <20050217190941.GA1561@vrfy.org> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: linux-hotplug@vger.kernel.org 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_id396&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