Browse Source

Fix part of #1963 (use ATTRS instead of SYSFS), add the note about the backslash at the end of each line back into the book, and add a reference to #1912 to the changelog entry from r7924.

git-svn-id: http://svn.linuxfromscratch.org/LFS/trunk/BOOK@7926 4aa44e1e-78dd-0310-a6d2-fbcd4c07a689
Bryan Kadzban 18 years ago
parent
commit
659cc3deea
2 changed files with 28 additions and 3 deletions
  1. 14 1
      chapter01/changelog.xml
  2. 14 2
      chapter07/symlinks.xml

+ 14 - 1
chapter01/changelog.xml

@@ -36,13 +36,26 @@
     </listitem>
     </listitem>
 -->
 -->
 
 
+    <listitem>
+      <para>2007-02-18</para>
+      <itemizedlist>
+        <listitem>
+          <para>[bryan] - Fix obsolete syntax used in the example Udev rules
+          in section 7.12.2. Fixes the easy part of
+          <ulink url="&lfs-ticket-root;1963">#1963</ulink>. Also re-add the
+          note about the backslash at the end of the udev rule lines.</para>
+        </listitem>
+      </itemizedlist>
+    </listitem>
+
     <listitem>
     <listitem>
       <para>2007-02-17</para>
       <para>2007-02-17</para>
       <itemizedlist>
       <itemizedlist>
         <listitem>
         <listitem>
           <para>[bryan] - Use upstream's rule_generator rules exclusively for
           <para>[bryan] - Use upstream's rule_generator rules exclusively for
           NIC naming, and generate the rules before configuring the network
           NIC naming, and generate the rules before configuring the network
-          script, so the user knows what NIC names to use.</para>
+          script, so the user knows what NIC names to use. Fixes the rest of
+          <ulink url="&lfs-ticket-root;1912">#1912</ulink>.</para>
         </listitem>
         </listitem>
         <listitem>
         <listitem>
           <para>[bryan] - Change from writing CD symlink rules files directly
           <para>[bryan] - Change from writing CD symlink rules files directly

+ 14 - 2
chapter07/symlinks.xml

@@ -122,13 +122,25 @@
 <screen role="nodump"><userinput>cat &gt; /etc/udev/rules.d/83-duplicate_devs.rules &lt;&lt; EOF
 <screen role="nodump"><userinput>cat &gt; /etc/udev/rules.d/83-duplicate_devs.rules &lt;&lt; EOF
 <literal>
 <literal>
 # Persistent symlinks for webcam and tuner
 # Persistent symlinks for webcam and tuner
-KERNEL=="video*", SYSFS{idProduct}=="1910", SYSFS{idVendor}=="0d81", \
+KERNEL=="video*", ATTRS{idProduct}=="1910", ATTRS{idVendor}=="0d81", \
     SYMLINK+="webcam"
     SYMLINK+="webcam"
-KERNEL=="video*", SYSFS{device}=="0x036f", SYSFS{vendor}=="0x109e", \
+KERNEL=="video*", ATTRS{device}=="0x036f", ATTRS{vendor}=="0x109e", \
     SYMLINK+="tvtuner"
     SYMLINK+="tvtuner"
 </literal>
 </literal>
 EOF</userinput></screen>
 EOF</userinput></screen>
 
 
+    <note>
+      <para>Be aware that Udev does not recognize the backslash for line
+      continuation.  This example works properly because both the backslash
+      and newline are ignored by the shell.  This makes the shell send each
+      rule to cat on only one line.  (The shell ignores this sequence because
+      the EOF string used in the here-document redirection is not enclosed in
+      either double or single quotes.  For more details, see the bash(1)
+      manpage, and search it for "Here Documents".)</para> 
+      <para>If modifying Udev rules with an editor, be sure to leave each
+      rule on one physical line.</para>
+    </note>
+
     <para>The result is that <filename>/dev/video0</filename> and
     <para>The result is that <filename>/dev/video0</filename> and
     <filename>/dev/video1</filename> devices still refer randomly to the tuner
     <filename>/dev/video1</filename> devices still refer randomly to the tuner
     and the web camera (and thus should never be used directly), but there are
     and the web camera (and thus should never be used directly), but there are