From ee66f55213b1fd5a006e8638d524e490c9f65f3b Mon Sep 17 00:00:00 2001 From: Eric Andersen Date: Fri, 7 Apr 2006 20:30:43 +0000 Subject: [PATCH] rework fakeroot handling to avoid races during parallel make causing several fakeroot instances stomping eachother's temp files. --- target/cramfs/cramfs.mk | 27 +++++++++++++++------------ target/ext2/ext2root.mk | 29 +++++++++++++++-------------- target/iso9660/iso9660.mk | 20 ++++++++++++++------ target/jffs2/jffs2root.mk | 27 +++++++++++++-------------- target/makedevs/makedevs.mk | 6 +----- target/squashfs/squashfsroot.mk | 27 +++++++++++++++------------ target/tar/tarroot.mk | 28 +++++++++++++++------------- 7 files changed, 88 insertions(+), 76 deletions(-) diff --git a/target/cramfs/cramfs.mk b/target/cramfs/cramfs.mk index 408ff3ac1b..1205c0030e 100644 --- a/target/cramfs/cramfs.mk +++ b/target/cramfs/cramfs.mk @@ -58,30 +58,33 @@ ifeq ($(strip $(BR2_sparc)),y) CRAMFS_ENDIANNESS=-b endif -cramfsroot: host-fakeroot makedevs $(STAGING_DIR)/fakeroot.env cramfs +CRAMFS_TARGET=$(IMAGE).cramfs + +cramfsroot: host-fakeroot makedevs cramfs #-@find $(TARGET_DIR)/lib -type f -name \*.so\* | xargs $(STRIP) --strip-unneeded 2>/dev/null || true; -@find $(TARGET_DIR) -type f -perm +111 | xargs $(STRIP) 2>/dev/null || true; @rm -rf $(TARGET_DIR)/usr/man @rm -rf $(TARGET_DIR)/usr/info -/sbin/ldconfig -r $(TARGET_DIR) 2>/dev/null # Use fakeroot to pretend all target binaries are owned by root + rm -f $(STAGING_DIR)/_fakeroot.$(CRAMFS_TARGET) + cat $(STAGING_DIR)/.fakeroot* > $(STAGING_DIR)/_fakeroot.$(CRAMFS_TARGET) -$(STAGING_DIR)/usr/bin/fakeroot \ - -i $(STAGING_DIR)/fakeroot.env \ - -s $(STAGING_DIR)/fakeroot.env -- \ + -i $(STAGING_DIR)/_fakeroot.$(CRAMFS_TARGET) \ + -s $(STAGING_DIR)/_fakeroot.$(CRAMFS_TARGET) -- \ chown -R root:root $(TARGET_DIR) # Use fakeroot to pretend to create all needed device nodes $(STAGING_DIR)/usr/bin/fakeroot \ - -i $(STAGING_DIR)/fakeroot.env \ - -s $(STAGING_DIR)/fakeroot.env -- \ - $(STAGING_DIR)/bin/makedevs \ - -d $(TARGET_DEVICE_TABLE) \ - $(TARGET_DIR) + -i $(STAGING_DIR)/_fakeroot.$(CRAMFS_TARGET) \ + -s $(STAGING_DIR)/_fakeroot.$(CRAMFS_TARGET) -- \ + $(STAGING_DIR)/bin/makedevs -d $(TARGET_DEVICE_TABLE) $(TARGET_DIR) # Use fakeroot so mkcramfs believes the previous fakery $(STAGING_DIR)/usr/bin/fakeroot \ - -i $(STAGING_DIR)/fakeroot.env \ - -s $(STAGING_DIR)/fakeroot.env -- \ - $(CRAMFS_DIR)/mkcramfs -q $(CRAMFS_ENDIANNESS) \ - $(TARGET_DIR) $(IMAGE).cramfs + -i $(STAGING_DIR)/_fakeroot.$(CRAMFS_TARGET) \ + -s $(STAGING_DIR)/_fakeroot.$(CRAMFS_TARGET) -- \ + $(CRAMFS_DIR)/mkcramfs -q $(CRAMFS_ENDIANNESS) \ + $(TARGET_DIR) $(CRAMFS_TARGET) + -@rm -f $(STAGING_DIR)/_fakeroot.$(CRAMFS_TARGET) cramfsroot-source: cramfs-source diff --git a/target/ext2/ext2root.mk b/target/ext2/ext2root.mk index fec8795b53..06ab3d2f3b 100644 --- a/target/ext2/ext2root.mk +++ b/target/ext2/ext2root.mk @@ -65,24 +65,24 @@ else EXT2_TARGET := $(EXT2_BASE) endif -$(EXT2_BASE): host-fakeroot makedevs $(STAGING_DIR)/fakeroot.env genext2fs +$(EXT2_BASE): host-fakeroot makedevs genext2fs -@find $(TARGET_DIR) -type f -perm +111 | xargs $(STRIP) 2>/dev/null || true; @rm -rf $(TARGET_DIR)/usr/man @rm -rf $(TARGET_DIR)/usr/share/man @rm -rf $(TARGET_DIR)/usr/info -/sbin/ldconfig -r $(TARGET_DIR) 2>/dev/null # Use fakeroot to pretend all target binaries are owned by root + rm -f $(STAGING_DIR)/_fakeroot.$(EXT2_TARGET) + cat $(STAGING_DIR)/.fakeroot* > $(STAGING_DIR)/_fakeroot.$(EXT2_TARGET) -$(STAGING_DIR)/usr/bin/fakeroot \ - -i $(STAGING_DIR)/fakeroot.env \ - -s $(STAGING_DIR)/fakeroot.env -- \ + -i $(STAGING_DIR)/_fakeroot.$(EXT2_TARGET) \ + -s $(STAGING_DIR)/_fakeroot.$(EXT2_TARGET) -- \ chown -R root:root $(TARGET_DIR) # Use fakeroot to pretend to create all needed device nodes $(STAGING_DIR)/usr/bin/fakeroot \ - -i $(STAGING_DIR)/fakeroot.env \ - -s $(STAGING_DIR)/fakeroot.env -- \ - $(STAGING_DIR)/bin/makedevs \ - -d $(TARGET_DEVICE_TABLE) \ - $(TARGET_DIR) + -i $(STAGING_DIR)/_fakeroot.$(EXT2_TARGET) \ + -s $(STAGING_DIR)/_fakeroot.$(EXT2_TARGET) -- \ + $(STAGING_DIR)/bin/makedevs -d $(TARGET_DEVICE_TABLE) $(TARGET_DIR) # Use fakeroot so genext2fs believes the previous fakery ifeq ($(strip $(BR2_TARGET_ROOTFS_EXT2_BLOCKS)),0) GENEXT2_REALSIZE=`LANG=C du -l -s -c -k $(TARGET_DIR) | grep total | sed -e "s/total//"`; \ @@ -92,22 +92,23 @@ ifeq ($(strip $(BR2_TARGET_ROOTFS_EXT2_BLOCKS)),0) GENEXT2_INODES=`expr $$GENEXT2_ADDTOINODESIZE + 400`; \ set -x; \ $(STAGING_DIR)/usr/bin/fakeroot \ - -i $(STAGING_DIR)/fakeroot.env \ - -s $(STAGING_DIR)/fakeroot.env -- \ - $(GENEXT2_DIR)/genext2fs \ + -i $(STAGING_DIR)/_fakeroot.$(EXT2_TARGET) \ + -s $(STAGING_DIR)/_fakeroot.$(EXT2_TARGET) -- \ + $(GENEXT2_DIR)/genext2fs \ -b $$GENEXT2_SIZE \ -i $$GENEXT2_INODES \ -d $(TARGET_DIR) \ $(EXT2_OPTS) $(EXT2_BASE) else $(STAGING_DIR)/usr/bin/fakeroot \ - -i $(STAGING_DIR)/fakeroot.env \ - -s $(STAGING_DIR)/fakeroot.env -- \ - $(GENEXT2_DIR)/genext2fs \ + -i $(STAGING_DIR)/_fakeroot.$(EXT2_TARGET) \ + -s $(STAGING_DIR)/_fakeroot.$(EXT2_TARGET) -- \ + $(GENEXT2_DIR)/genext2fs \ -d $(TARGET_DIR) \ $(EXT2_OPTS) \ $(EXT2_BASE) endif + -@rm -f $(STAGING_DIR)/_fakeroot.$(EXT2_TARGET) $(EXT2_BASE).gz: $(EXT2_BASE) @gzip --best -fv $(EXT2_BASE) diff --git a/target/iso9660/iso9660.mk b/target/iso9660/iso9660.mk index a0b074000c..1bc415aed5 100644 --- a/target/iso9660/iso9660.mk +++ b/target/iso9660/iso9660.mk @@ -52,22 +52,30 @@ ifeq ($(strip $(BR2_TARGET_ROOTFS_ISO9660_SQUASH)),y) ISO9660_OPTS+=-U endif -$(ISO9660_TARGET): host-fakeroot $(STAGING_DIR)/fakeroot.env $(EXT2_TARGET) grub mkisofs +$(ISO9660_TARGET): host-fakeroot $(EXT2_TARGET) grub mkisofs mkdir -p $(ISO9660_TARGET_DIR) mkdir -p $(ISO9660_TARGET_DIR)/boot/grub cp $(GRUB_DIR)/stage2/stage2_eltorito $(ISO9660_TARGET_DIR)/boot/grub/ cp $(ISO9660_BOOT_MENU) $(ISO9660_TARGET_DIR)/boot/grub/menu.lst cp $(LINUX_KERNEL) $(ISO9660_TARGET_DIR)/kernel cp $(EXT2_TARGET) $(ISO9660_TARGET_DIR)/initrd + # Use fakeroot to pretend all target binaries are owned by root + rm -f $(STAGING_DIR)/_fakeroot.$(ISO9660_TARGET) + cat $(STAGING_DIR)/.fakeroot* > $(STAGING_DIR)/_fakeroot.$(ISO9660_TARGET) -$(STAGING_DIR)/usr/bin/fakeroot \ - -i $(STAGING_DIR)/fakeroot.env \ - -s $(STAGING_DIR)/fakeroot.env -- \ + -i $(STAGING_DIR)/_fakeroot.$(ISO9660_TARGET) \ + -s $(STAGING_DIR)/_fakeroot.$(ISO9660_TARGET) -- \ chown -R root:root $(ISO9660_TARGET_DIR) - -$(MKISOFS_TARGET) \ - -R \ - -b boot/grub/stage2_eltorito -no-emul-boot -boot-load-size 4 -boot-info-table \ + # Use fakeroot so mkisofs believes the previous fakery + $(STAGING_DIR)/usr/bin/fakeroot \ + -i $(STAGING_DIR)/_fakeroot.$(ISO9660_TARGET) \ + -s $(STAGING_DIR)/_fakeroot.$(ISO9660_TARGET) -- \ + $(MKISOFS_TARGET) \ + -R -b boot/grub/stage2_eltorito -no-emul-boot \ + -boot-load-size 4 -boot-info-table \ -o $(ISO9660_TARGET) \ $(ISO9660_TARGET_DIR) + -@rm -f $(STAGING_DIR)/_fakeroot.$(ISO9660_TARGET) iso9660root: $(ISO9660_TARGET) echo $(ISO9660_TARGET) diff --git a/target/jffs2/jffs2root.mk b/target/jffs2/jffs2root.mk index c5efae5254..2c6ad5eb75 100644 --- a/target/jffs2/jffs2root.mk +++ b/target/jffs2/jffs2root.mk @@ -38,32 +38,31 @@ JFFS2_TARGET := $(subst ",,$(BR2_TARGET_ROOTFS_JFFS2_OUTPUT)) # value of MKFS_JFFS2 to either the previously installed copy or the one # just built. # -$(JFFS2_TARGET): host-fakeroot makedevs $(STAGING_DIR)/fakeroot.env mtd-host +$(JFFS2_TARGET): host-fakeroot makedevs mtd-host -@find $(TARGET_DIR) -type f -perm +111 | xargs $(STRIP) 2>/dev/null || true; @rm -rf $(TARGET_DIR)/usr/man @rm -rf $(TARGET_DIR)/usr/share/man @rm -rf $(TARGET_DIR)/usr/info -/sbin/ldconfig -r $(TARGET_DIR) 2>/dev/null # Use fakeroot to pretend all target binaries are owned by root + rm -f $(STAGING_DIR)/_fakeroot.$(JFFS2_TARGET) + cat $(STAGING_DIR)/.fakeroot* > $(STAGING_DIR)/_fakeroot.$(JFFS2_TARGET) -$(STAGING_DIR)/usr/bin/fakeroot \ - -i $(STAGING_DIR)/fakeroot.env \ - -s $(STAGING_DIR)/fakeroot.env -- \ + -i $(STAGING_DIR)/_fakeroot.$(JFFS2_TARGET) \ + -s $(STAGING_DIR)/_fakeroot.$(JFFS2_TARGET) -- \ chown -R root:root $(TARGET_DIR) # Use fakeroot to pretend to create all needed device nodes $(STAGING_DIR)/usr/bin/fakeroot \ - -i $(STAGING_DIR)/fakeroot.env \ - -s $(STAGING_DIR)/fakeroot.env -- \ - $(STAGING_DIR)/bin/makedevs \ - -d $(TARGET_DEVICE_TABLE) \ - $(TARGET_DIR) + -i $(STAGING_DIR)/_fakeroot.$(JFFS2_TARGET) \ + -s $(STAGING_DIR)/_fakeroot.$(JFFS2_TARGET) -- \ + $(STAGING_DIR)/bin/makedevs -d $(TARGET_DEVICE_TABLE) $(TARGET_DIR) # Use fakeroot so mkfs.jffs2 believes the previous fakery $(STAGING_DIR)/usr/bin/fakeroot \ - -i $(STAGING_DIR)/fakeroot.env \ - -s $(STAGING_DIR)/fakeroot.env -- \ - $(MKFS_JFFS2) \ - $(JFFS2_OPTS) \ - -d $(BUILD_DIR)/root \ - -o $(JFFS2_TARGET) + -i $(STAGING_DIR)/_fakeroot.$(JFFS2_TARGET) \ + -s $(STAGING_DIR)/_fakeroot.$(JFFS2_TARGET) -- \ + $(MKFS_JFFS2) $(JFFS2_OPTS) \ + -d $(BUILD_DIR)/root -o $(JFFS2_TARGET) + -@rm -f $(STAGING_DIR)/_fakeroot.$(JFFS2_TARGET) @ls -l $(JFFS2_TARGET) JFFS2_COPYTO := $(strip $(subst ",,$(BR2_TARGET_ROOTFS_JFFS2_COPYTO))) diff --git a/target/makedevs/makedevs.mk b/target/makedevs/makedevs.mk index 334f65fa25..620b0cdcd6 100644 --- a/target/makedevs/makedevs.mk +++ b/target/makedevs/makedevs.mk @@ -13,17 +13,13 @@ $(MAKEDEVS_DIR)/makedevs.c: target/makedevs/makedevs.c cp target/makedevs/makedevs.c $(MAKEDEVS_DIR) $(MAKEDEVS_DIR)/makedevs: $(MAKEDEVS_DIR)/makedevs.c - $(HOSTCC) -Wall -Werror -O2 $(MAKEDEVS_DIR)/makedevs.c -o $(MAKEDEVS_DIR)/makedevs + gcc -Wall -Werror -O2 $(MAKEDEVS_DIR)/makedevs.c -o $(MAKEDEVS_DIR)/makedevs touch -c $(MAKEDEVS_DIR)/makedevs $(STAGING_DIR)/bin/makedevs: $(MAKEDEVS_DIR)/makedevs $(INSTALL) -m 755 $(MAKEDEVS_DIR)/makedevs $(STAGING_DIR)/bin/makedevs touch -c $(STAGING_DIR)/bin/makedevs -$(STAGING_DIR)/fakeroot.env: - cat $(STAGING_DIR)/.fakeroot.* > $(STAGING_DIR)/fakeroot.env - touch -c $(STAGING_DIR)/fakeroot.env - makedevs: $(STAGING_DIR)/bin/makedevs makedevs-source: diff --git a/target/squashfs/squashfsroot.mk b/target/squashfs/squashfsroot.mk index 4a2ff2880a..10f0c5205a 100644 --- a/target/squashfs/squashfsroot.mk +++ b/target/squashfs/squashfsroot.mk @@ -53,31 +53,34 @@ ifeq ($(strip $(BR2_sparc)),y) SQUASHFS_ENDIANNESS=-be endif -squashfsroot: host-fakeroot makedevs $(STAGING_DIR)/fakeroot.env squashfs +SQUASHFS_TARGET:=$(IMAGE).squashfs + +squashfsroot: host-fakeroot makedevs squashfs -@find $(TARGET_DIR) -type f -perm +111 | xargs $(STRIP) 2>/dev/null || true; @rm -rf $(TARGET_DIR)/usr/man @rm -rf $(TARGET_DIR)/usr/info -/sbin/ldconfig -r $(TARGET_DIR) 2>/dev/null # Use fakeroot to pretend all target binaries are owned by root + rm -f $(STAGING_DIR)/_fakeroot.$(SQUASHFS_TARGET) + cat $(STAGING_DIR)/.fakeroot* > $(STAGING_DIR)/_fakeroot.$(SQUASHFS_TARGET) -$(STAGING_DIR)/usr/bin/fakeroot \ - -i $(STAGING_DIR)/fakeroot.env \ - -s $(STAGING_DIR)/fakeroot.env -- \ + -i $(STAGING_DIR)/_fakeroot.$(SQUASHFS_TARGET) \ + -s $(STAGING_DIR)/_fakeroot.$(SQUASHFS_TARGET) -- \ chown -R root:root $(TARGET_DIR) # Use fakeroot to pretend to create all needed device nodes $(STAGING_DIR)/usr/bin/fakeroot \ - -i $(STAGING_DIR)/fakeroot.env \ - -s $(STAGING_DIR)/fakeroot.env -- \ - $(STAGING_DIR)/bin/makedevs \ - -d $(TARGET_DEVICE_TABLE) \ - $(TARGET_DIR) + -i $(STAGING_DIR)/_fakeroot.$(SQUASHFS_TARGET) \ + -s $(STAGING_DIR)/_fakeroot.$(SQUASHFS_TARGET) -- \ + $(STAGING_DIR)/bin/makedevs -d $(TARGET_DEVICE_TABLE) $(TARGET_DIR) # Use fakeroot so mksquashfs believes the previous fakery $(STAGING_DIR)/usr/bin/fakeroot \ - -i $(STAGING_DIR)/fakeroot.env \ - -s $(STAGING_DIR)/fakeroot.env -- \ - $(SQUASHFS_DIR)/squashfs-tools/mksquashfs \ + -i $(STAGING_DIR)/_fakeroot.$(SQUASHFS_TARGET) \ + -s $(STAGING_DIR)/_fakeroot.$(SQUASHFS_TARGET) -- \ + $(SQUASHFS_DIR)/squashfs-tools/mksquashfs \ $(TARGET_DIR) \ - $(IMAGE).squashfs \ + $(SQUASHFS_TARGET) \ -noappend $(SQUASHFS_ENDIANNESS) + -@rm -f $(STAGING_DIR)/_fakeroot.$(SQUASHFS_TARGET) squashfsroot-source: squashfs-source diff --git a/target/tar/tarroot.mk b/target/tar/tarroot.mk index 867adc1b92..cbf538653c 100644 --- a/target/tar/tarroot.mk +++ b/target/tar/tarroot.mk @@ -4,30 +4,32 @@ # ############################################################# -TAR_OPTS := $(strip $(subst ",, $(BR2_TARGET_ROOTFS_TAR_OPTIONS))) -#" -tarroot: host-fakeroot makedevs $(STAGING_DIR)/fakeroot.env +TAR_OPTS:=$(strip $(BR2_TARGET_ROOTFS_TAR_OPTIONS)) +TAR_TARGET:=$(IMAGE).tar + +tarroot: host-fakeroot makedevs -@find $(TARGET_DIR) -type f -perm +111 | xargs $(STRIP) 2>/dev/null || true; @rm -rf $(TARGET_DIR)/usr/man @rm -rf $(TARGET_DIR)/usr/info -/sbin/ldconfig -r $(TARGET_DIR) 2>/dev/null # Use fakeroot to pretend all target binaries are owned by root + rm -f $(STAGING_DIR)/_fakeroot.$(TAR_TARGET) + cat $(STAGING_DIR)/.fakeroot* > $(STAGING_DIR)/_fakeroot.$(TAR_TARGET) -$(STAGING_DIR)/usr/bin/fakeroot \ - -i $(STAGING_DIR)/fakeroot.env \ - -s $(STAGING_DIR)/fakeroot.env -- \ + -i $(STAGING_DIR)/_fakeroot.$(TAR_TARGET) \ + -s $(STAGING_DIR)/_fakeroot.$(TAR_TARGET) -- \ chown -R root:root $(TARGET_DIR) # Use fakeroot to pretend to create all needed device nodes $(STAGING_DIR)/usr/bin/fakeroot \ - -i $(STAGING_DIR)/fakeroot.env \ - -s $(STAGING_DIR)/fakeroot.env -- \ - $(STAGING_DIR)/bin/makedevs \ - -d $(TARGET_DEVICE_TABLE) \ - $(TARGET_DIR) + -i $(STAGING_DIR)/_fakeroot.$(TAR_TARGET) \ + -s $(STAGING_DIR)/_fakeroot.$(TAR_TARGET) -- \ + $(STAGING_DIR)/bin/makedevs -d $(TARGET_DEVICE_TABLE) $(TARGET_DIR) # Use fakeroot so tar believes the previous fakery $(STAGING_DIR)/usr/bin/fakeroot \ - -i $(STAGING_DIR)/fakeroot.env \ - -s $(STAGING_DIR)/fakeroot.env -- \ - tar -c$(TAR_OPTS)f $(IMAGE).tar -C $(TARGET_DIR) . + -i $(STAGING_DIR)/_fakeroot.$(TAR_TARGET) \ + -s $(STAGING_DIR)/_fakeroot.$(TAR_TARGET) -- \ + tar -c$(TAR_OPTS)f $(TAR_TARGET) -C $(TARGET_DIR) . + -@rm -f $(STAGING_DIR)/_fakeroot.$(TAR_TARGET) tarroot-source: -- 2.30.2