linux-hotplug.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Martin Hicks <mort@bork.org>
To: linux-hotplug@vger.kernel.org
Subject: Re: [PATCH] udev - small script optimization
Date: Tue, 13 Jan 2004 19:27:23 +0000	[thread overview]
Message-ID: <20040113192723.GG30627@localhost> (raw)
In-Reply-To: <20040113035015.GA9989@vrfy.org>



On Tue, Jan 13, 2004 at 04:50:15AM +0100, Kay Sievers wrote:
> 
>   The name_cdrom.pl is only executed for ide and scsi devices, but not for a partition.
>   It exits with nonzero to skip the rule if the CD is not found:
> 
>     KERNEL="[hs]d[a-z]", PROGRAM="name_cdrom.pl %M %m", NAME="%1c", SYMLINK="cdrom"
> 

This won't work on machines with lots of SCSI targets (I suppose
IDE could have the same problem although it's much less likely)

I'm pretty sure that we don't support the '+' metacharacter, but what
would really work is:

KERNEL="[hs]d[a-z]+"

because after sdz we go on to sdaa.  I'm not really complaining, because
it seems unlikely that you would find machines with cdroms after the
first 26 SCSI targets.  I'm just pointing out that this rule isn't
exactly correct.

mh

-- 
Martin Hicks || mort@bork.org || PGP/GnuPG: 0x4C7F2BEE


-------------------------------------------------------
This SF.net email is sponsored by: Perforce Software.
Perforce is the Fast Software Configuration Management System offering
advanced branching capabilities and atomic changes on 50+ platforms.
Free Eval! http://www.perforce.com/perforce/loadprog.html
_______________________________________________
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

      parent reply	other threads:[~2004-01-13 19:27 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-01-13  3:50 [PATCH] udev - small script optimization Kay Sievers
2004-01-13 18:42 ` Greg KH
2004-01-13 19:27 ` Martin Hicks [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20040113192723.GG30627@localhost \
    --to=mort@bork.org \
    --cc=linux-hotplug@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).