Changeset 088130f

Timestamp:

09/17/2020 03:30:29 PM (4 years ago)

Author:

Pierre Labastie <pierre@…>

Branches:

ablfs-more, legacy, trunk

Children:

5562bf1

Parents:

426e40b

Message:

Fix changing the ownership of $BUILDDIR/var/lib to LUSER when it has been
created by func_install_blfs. Fixes ticket #1729

Files:

• LFS/master.sh (modified) (3 diffs)
• common/libs/func_install_blfs (modified) (1 diff)

LFS/master.sh

@@ -71 +71 @@
     esac
     # Grab the name of the target
-    # This is only use to check the name in "opt_override" or "BLACKIST"
+    # This is only used to check the name in "opt_override" or "BLACKIST"
     name=`echo ${this_script} | sed -e 's@[0-9]\{3,4\}-@@' \
                                     -e 's@-pass[0-9]\{1\}@@' \
@@ -143 +143 @@
       *addinguser)
 (
+# /var/lib may already exist and be owned by root if blfs tools
+# have been installed.
 cat << EOF
         @if [ -f luser-id ]; then \\
@@ -153 +155 @@
         \$(PRT_DU) >>logs/\$@
         @chown \$(LUSER):\$(LGROUP) envars
+        @[ -d "\$(MOUNT_PT)/var/lib" ] && \\
+            chown \$(LUSER):\$(LGROUP) \$(MOUNT_PT)/var/lib
         @chmod -R a+wt $JHALFSDIR
         @chmod a+wt \$(SRCSDIR)

common/libs/func_install_blfs

@@ -63 +63 @@
 
 # But then $BUILDDIR/var/lib/ is owned by root if just created, which
-# prevents $LUSER to create "nss_db". Change ownership here
-sudo chown $LUSER:$LGROUP $BUILDDIR/var/lib
+# prevents $LUSER to create "nss_db". The problem is that $LUSER
+# May not have been created yet. So do not change ownership here
+# and do it in master.sh for chapter 4.
+#sudo chown $LUSER:$LGROUP $BUILDDIR/var/lib
 
 # Because the BLFS Makefile is supposed to be used in chroot (or booted)