All of lore.kernel.org
 help / color / mirror / Atom feed
* IGNORE method
@ 2004-01-10  1:31 Kay Sievers
  2004-01-10  1:35 ` Greg KH
                   ` (3 more replies)
  0 siblings, 4 replies; 5+ messages in thread
From: Kay Sievers @ 2004-01-10  1:31 UTC (permalink / raw)
  To: linux-hotplug


http://linuxusb.bkbits.net:8080/udev/cset@1.395?nav=index.html|ChangeSet@-2d

We have a IGNORE rule now?
I would prefer a change that simply allows NAME="".
That works with every other method, not only kernel names and is much
much simpler.

Hmm...

Kay



-------------------------------------------------------
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

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: IGNORE method
  2004-01-10  1:31 IGNORE method Kay Sievers
@ 2004-01-10  1:35 ` Greg KH
  2004-01-10 10:44 ` Christophe Saout
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 5+ messages in thread
From: Greg KH @ 2004-01-10  1:35 UTC (permalink / raw)
  To: linux-hotplug

On Sat, Jan 10, 2004 at 02:31:28AM +0100, Kay Sievers wrote:
> 
> http://linuxusb.bkbits.net:8080/udev/cset@1.395?nav=index.html|ChangeSet@-2d
> 
> We have a IGNORE rule now?

Yeah, Christophe sent me a patch for one.

> I would prefer a change that simply allows NAME="".
> That works with every other method, not only kernel names and is much
> much simpler.

Ah.  Yeah, that might be a lot simpler.  Something like (in your new
syntax):
	KERNEL="dm-[0-9]*", NAME=""

It's not the most intuitive, but I could live with it.  Christophe, what
do you think?

thanks,

greg k-h


-------------------------------------------------------
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

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: IGNORE method
  2004-01-10  1:31 IGNORE method Kay Sievers
  2004-01-10  1:35 ` Greg KH
@ 2004-01-10 10:44 ` Christophe Saout
  2004-01-13 18:08 ` Greg KH
  2004-01-13 18:24 ` Christophe Saout
  3 siblings, 0 replies; 5+ messages in thread
From: Christophe Saout @ 2004-01-10 10:44 UTC (permalink / raw)
  To: linux-hotplug

Am Sa, den 10.01.2004 schrieb Greg KH um 02:35:
> On Sat, Jan 10, 2004 at 02:31:28AM +0100, Kay Sievers wrote:
> > 
> > http://linuxusb.bkbits.net:8080/udev/cset@1.395?nav=index.html|ChangeSet@-2d
> > 
> > We have a IGNORE rule now?
> 
> Yeah, Christophe sent me a patch for one.
> 
> > I would prefer a change that simply allows NAME="".
> > That works with every other method, not only kernel names and is much
> > much simpler.
> 
> Ah.  Yeah, that might be a lot simpler.  Something like (in your new
> syntax):
> 	KERNEL="dm-[0-9]*", NAME=""
> 
> It's not the most intuitive, but I could live with it.  Christophe, what
> do you think?

Sure, I thought of this too. I added the IGNORE rule because it's more
intuitive but if this is more flexible, I'm ok with it. You should be
aware that this would also ignore an additional SYMLINK.

Should I update my patch or add one on top of it that drops the new
rule?




-------------------------------------------------------
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

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: IGNORE method
  2004-01-10  1:31 IGNORE method Kay Sievers
  2004-01-10  1:35 ` Greg KH
  2004-01-10 10:44 ` Christophe Saout
@ 2004-01-13 18:08 ` Greg KH
  2004-01-13 18:24 ` Christophe Saout
  3 siblings, 0 replies; 5+ messages in thread
From: Greg KH @ 2004-01-13 18:08 UTC (permalink / raw)
  To: linux-hotplug

On Sat, Jan 10, 2004 at 11:44:10AM +0100, Christophe Saout wrote:
> Am Sa, den 10.01.2004 schrieb Greg KH um 02:35:
> > On Sat, Jan 10, 2004 at 02:31:28AM +0100, Kay Sievers wrote:
> > > 
> > > http://linuxusb.bkbits.net:8080/udev/cset@1.395?nav=index.html|ChangeSet@-2d
> > > 
> > > We have a IGNORE rule now?
> > 
> > Yeah, Christophe sent me a patch for one.
> > 
> > > I would prefer a change that simply allows NAME="".
> > > That works with every other method, not only kernel names and is much
> > > much simpler.
> > 
> > Ah.  Yeah, that might be a lot simpler.  Something like (in your new
> > syntax):
> > 	KERNEL="dm-[0-9]*", NAME=""
> > 
> > It's not the most intuitive, but I could live with it.  Christophe, what
> > do you think?
> 
> Sure, I thought of this too. I added the IGNORE rule because it's more
> intuitive but if this is more flexible, I'm ok with it. You should be
> aware that this would also ignore an additional SYMLINK.

So you would like to not create a name, but create a symlink?  What
would you create the symlink to?

> Should I update my patch or add one on top of it that drops the new
> rule?

Kay's patch took care of it already.

thanks,

greg k-h


-------------------------------------------------------
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

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: IGNORE method
  2004-01-10  1:31 IGNORE method Kay Sievers
                   ` (2 preceding siblings ...)
  2004-01-13 18:08 ` Greg KH
@ 2004-01-13 18:24 ` Christophe Saout
  3 siblings, 0 replies; 5+ messages in thread
From: Christophe Saout @ 2004-01-13 18:24 UTC (permalink / raw)
  To: linux-hotplug

Am Di, den 13.01.2004 schrieb Greg KH um 19:08:
> On Sat, Jan 10, 2004 at 11:44:10AM +0100, Christophe Saout wrote:
> > Am Sa, den 10.01.2004 schrieb Greg KH um 02:35:
> > >
> > > Ah.  Yeah, that might be a lot simpler.  Something like (in your new
> > > syntax):
> > > 	KERNEL="dm-[0-9]*", NAME=""
> > > 
> > > It's not the most intuitive, but I could live with it.  Christophe, what
> > > do you think?
> > 
> > Sure, I thought of this too. I added the IGNORE rule because it's more
> > intuitive but if this is more flexible, I'm ok with it. You should be
> > aware that this would also ignore an additional SYMLINK.
> 
> So you would like to not create a name, but create a symlink?  What
> would you create the symlink to?

No, I don't want to. :D I just wondered what should be done if a symlink
is specified but an empty name. A warning? At the moment udev silently
ignores errors anyway so... not important.




-------------------------------------------------------
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

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2004-01-13 18:24 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2004-01-10  1:31 IGNORE method Kay Sievers
2004-01-10  1:35 ` Greg KH
2004-01-10 10:44 ` Christophe Saout
2004-01-13 18:08 ` Greg KH
2004-01-13 18:24 ` Christophe Saout

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.