From 94dc9c26d2ee84c7d81b785d2d9124d630e2ca17 Mon Sep 17 00:00:00 2001
From: Xi Ruoyao <xry111@xry111.site>
Date: Sat, 8 Feb 2025 08:42:47 +0800
Subject: [PATCH 1/3] Revert "Fix cmds installing python docs (permissions)"

This reverts commit dc8572f2413a4d25444bb1237ea2a01dfba60429.

The permission issue seems fixed upstream: after I untar the tarball
with --no-same-owner --no-same-permissions as root, all files are 644
and all directories are 755 (proven via find -type f -not -perm 644 and
find -type d -not -perm 755).
---
 chapter08/python.xml | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/chapter08/python.xml b/chapter08/python.xml
index 75f18945f..ebda61181 100644
--- a/chapter08/python.xml
+++ b/chapter08/python.xml
@@ -162,21 +162,21 @@ EOF
 
 <screen><userinput remap="install">install -v -dm755 /usr/share/doc/python-&python-version;/html
 
-tar --no-same-owner \
-    -xvf ../python-&python-version;-docs-html.tar.bz2
-cp -R --no-preserve=mode python-&python-version;-docs-html/* \
-    /usr/share/doc/python-&python-version;/html</userinput></screen>
+tar --strip-components=1  \
+    --no-same-owner       \
+    --no-same-permissions \
+    -C /usr/share/doc/python-&python-version;/html \
+    -xvf ../python-&python-version;-docs-html.tar.bz2</userinput></screen>
 
     <variablelist>
       <title>The meaning of the documentation install commands:</title>
 
     <varlistentry>
-      <term><option>--no-same-owner</option> (tar) and <option>--no-preserve=mode</option> (cp)</term>
+      <term><option>--no-same-owner</option> and <option>--no-same-permissions</option></term>
       <listitem>
         <para>Ensure the installed files have the correct ownership and
         permissions.  Without these options, <application>tar</application>
-        will install the package files with the upstream creator's values
-        and files would have restrictive permissions.
+        will install the package files with the upstream creator's values.
         </para>
       </listitem>
     </varlistentry>

From 3f2235c907b373df5ae9512e1e9ada97443d9427 Mon Sep 17 00:00:00 2001
From: Xi Ruoyao <xry111@xry111.site>
Date: Sat, 8 Feb 2025 09:17:21 +0800
Subject: [PATCH 2/3] coreutils: Better explain the autoreconf and automake
 commands

---
 chapter08/coreutils.xml | 18 ++++++++++++------
 1 file changed, 12 insertions(+), 6 deletions(-)

diff --git a/chapter08/coreutils.xml b/chapter08/coreutils.xml
index 70eb1e318..1acad31ca 100644
--- a/chapter08/coreutils.xml
+++ b/chapter08/coreutils.xml
@@ -62,22 +62,28 @@ FORCE_UNSAFE_CONFIGURE=1 ./configure \
             --enable-no-install-program=kill,uptime</userinput></screen>
 
     <variablelist>
-      <title>The meaning of the configure options:</title>
+      <title>The meaning of the commands and configure options:</title>
 
       <varlistentry>
-        <term><command>autoreconf</command></term>
+        <term><command>autoreconf -fv</command></term>
         <listitem>
           <para>The patch for internationalization has modified the
           build system, so the configuration files must
-          be regenerated.</para>
+          be regenerated.  Normally we would use the
+          <parameter>-i</parameter> option to update the standard
+          auxilary files, but for this package it does not work because
+          <filename>configure.ac</filename> specified an old gettext
+          version.</para>
         </listitem>
       </varlistentry>
 
       <varlistentry>
-        <term><command>automake</command></term>
+        <term><command>automake -af</command></term>
         <listitem>
-          <para>Work around a problem with an older versions of autotools.
-          </para>
+	  <para>The automake auxilary files were not updated by
+	  <command>autoreconf</command> due to the missing
+	  <parameter>-i</parameter> option.  This command updates them
+	  to prevent a build failure.</para>
         </listitem>
       </varlistentry>
 

From cb1b56efce6ba83f916bcb5da854045f10054ad5 Mon Sep 17 00:00:00 2001
From: Xi Ruoyao <xry111@xry111.site>
Date: Sat, 8 Feb 2025 09:50:39 +0800
Subject: [PATCH 3/3] coreutils: Remove outdated known failures

They seem fixed with 9.6.
---
 chapter08/coreutils.xml | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/chapter08/coreutils.xml b/chapter08/coreutils.xml
index 1acad31ca..dfe6d0da5 100644
--- a/chapter08/coreutils.xml
+++ b/chapter08/coreutils.xml
@@ -149,12 +149,6 @@ FORCE_UNSAFE_CONFIGURE=1 ./configure \
 
 <screen><userinput remap="test">groupdel dummy</userinput></screen>
 
-    <para>
-       Two tests, <filename>tests/cp/preserve-mode.sh</filename> and
-       <filename>tests/mv/acl.sh</filename>, are known to
-       fail in the chroot environment, but pass in a complete system.
-    </para>
-
     <para>Install the package:</para>
 
 <screen><userinput remap="install">make install</userinput></screen>