Patch to udev.7 to clarify $attr{} parent searching

Patch to udev.7 to clarify $attr{} parent searching

[Karl O. Pinc]

[-- Attachment #1: Type: text/plain, Size: 449 bytes --]

Attached patch against udev.7 of udev-124.

Clarifies that the first match is used when
$attr{} substitution searches through parent
devices.

Apply in udev-124 directory with:
patch < /foo/udev.7-patch

(Also, I didn't make the change, but shouldn't
the last sentence of the patched paragraph be
a new paragraph?)


Karl <kop@meme.com>
Free Software:  "You don't pay back, you pay forward."
                  -- Robert A. Heinlein

[-- Attachment #2: udev.7-patch --]
[-- Type: text/x-patch, Size: 973 bytes --]

--- ../../udev-124/udev.7	2008-06-12 00:24:30.000000000 -0500
+++ udev.7	2008-06-27 19:28:18.000000000 -0500
@@ -357,7 +357,7 @@
 .PP
 \fB$attr{\fR\fB\fIfile\fR\fR\fB}\fR, \fB%s{\fR\fB\fIfile\fR\fR\fB}\fR
 .RS 4
-The value of a sysfs attribute found at the device, where all keys of the rule have matched\. If the matching device does not have such an attribute, all devices along the chain of parents are searched for a matching attribute\. If the attribute is a symlink, the last element of the symlink target is returned as the value\.
+The value of a sysfs attribute found at the device, where all keys of the rule have matched\. If the matching device does not have such an attribute then follow the chain of parent devices and use the value of the first attribute that matches\. If the attribute is a symlink, the last element of the symlink target is returned as the value\.
 .RE
 .PP
 \fB$env{\fR\fB\fIkey\fR\fR\fB}\fR, \fB%E{\fR\fB\fIkey\fR\fR\fB}\fR

Re: Patch to udev.7 to clarify $attr{} parent searching

[Kay Sievers]

On Sat, Jun 28, 2008 at 02:38, Karl O. Pinc <kop@meme.com> wrote:
> Attached patch against udev.7 of udev-124.
>
> Clarifies that the first match is used when
> $attr{} substitution searches through parent
> devices.
>
> Apply in udev-124 directory with:
> patch < /foo/udev.7-patch
>
> (Also, I didn't make the change, but shouldn't
> the last sentence of the patched paragraph be
> a new paragraph?)

Applied the patch. If possible, please send patches against the XML
source, where the man page is built from, in the udev tree.

Thanks,
Kay

Re: Patch to udev.7 to clarify $attr{} parent searching

[Karl O. Pinc]

On 07/02/2008 03:22:51 AM, Kay Sievers wrote:

> If possible, please send patches against the XML
> source, where the man page is built from, in the udev tree.

Oops.  Sorry.

I just figured out there's a git tree.  I don't know squat
about git.  Is git://git.kernel.org/pub/scm/linux/hotplug/udev
the right url to copy the repository?  (This is a wild guess,
I've not figured out how to browse a tree.)

Karl <kop@meme.com>
Free Software:  "You don't pay back, you pay forward."
                  -- Robert A. Heinlein

Re: Patch to udev.7 to clarify $attr{} parent searching

[Kay Sievers]

On Wed, Jul 2, 2008 at 18:00, Karl O. Pinc <kop@meme.com> wrote:
>
> On 07/02/2008 03:22:51 AM, Kay Sievers wrote:
>
>> If possible, please send patches against the XML
>> source, where the man page is built from, in the udev tree.
>
> Oops.  Sorry.
>
> I just figured out there's a git tree.  I don't know squat
> about git.  Is git://git.kernel.org/pub/scm/linux/hotplug/udev
> the right url to copy the repository?  (This is a wild guess,
> I've not figured out how to browse a tree.)

Yes, it's here:
  http://git.kernel.org/?p=linux/hotplug/udev.git;a=summary

Kay