[Buildroot] [PATCH 02/12] fs/iso9660: convert to the filesystem infrastructure
Samuel Martin
s.martin49 at gmail.com
Fri Jun 5 11:59:50 UTC 2015
Hi Thomas,
On Thu, Jun 4, 2015 at 5:05 PM, Thomas Petazzoni
<thomas.petazzoni at free-electrons.com> wrote:
> Now that we have <fs>_POST_GEN_HOOKS, there is nothing that prevents
> the iso9660 filesystem logic from using the common filesystem
> infrastructure.
>
> Signed-off-by: Thomas Petazzoni <thomas.petazzoni at free-electrons.com>
> ---
> fs/iso9660/iso9660.mk | 64 +++++++++++++++++++++++++--------------------------
> 1 file changed, 31 insertions(+), 33 deletions(-)
>
> diff --git a/fs/iso9660/iso9660.mk b/fs/iso9660/iso9660.mk
> index 4ccfce9..5a29673 100644
> --- a/fs/iso9660/iso9660.mk
> +++ b/fs/iso9660/iso9660.mk
> @@ -2,9 +2,6 @@
> #
> # Build the iso96600 root filesystem image
> #
> -# Cannot be converted to the ROOTFS_TARGET infrastructure, because of
> -# the temporary construction in ISO9660_TARGET_DIR.
> -#
> ################################################################################
>
> ISO9660_TARGET_DIR = $(BUILD_DIR)/iso9660
> @@ -12,49 +9,50 @@ ISO9660_BOOT_MENU := $(call qstrip,$(BR2_TARGET_ROOTFS_ISO9660_BOOT_MENU))
>
> ROOTFS_ISO9660_DEPENDENCIES = grub host-cdrkit host-fakeroot linux rootfs-cpio
>
> -$(BINARIES_DIR)/rootfs.iso9660: $(ROOTFS_ISO9660_DEPENDENCIES)
> - @$(call MESSAGE,"Generating root filesystem image rootfs.iso9660")
> - $(INSTALL) -D -m 0644 $(GRUB_DIR)/stage2/stage2_eltorito \
> - $(ISO9660_TARGET_DIR)/boot/grub/stage2_eltorito
> - $(INSTALL) -D -m 0644 $(ISO9660_BOOT_MENU) \
> - $(ISO9660_TARGET_DIR)/boot/grub/menu.lst
> ifeq ($(BR2_TARGET_GRUB_SPLASH),)
> +define ROOTFS_ISO9660_SPLASHSCREEN
> $(SED) '/^splashimage/d' $(ISO9660_TARGET_DIR)/boot/grub/menu.lst
> +endef
> else
> +define ROOTFS_ISO9660_SPLASHSCREEN
> $(INSTALL) -D -m 0644 boot/grub/splash.xpm.gz \
> $(ISO9660_TARGET_DIR)/splash.xpm.gz
> +endef
> endif
> - $(INSTALL) -D -m 0644 $(LINUX_IMAGE_PATH) $(ISO9660_TARGET_DIR)/kernel
> +
> ifeq ($(BR2_TARGET_ROOTFS_INITRAMFS),y)
> +define ROOTFS_ISO9660_INITRD
> $(SED) '/initrd/d' $(ISO9660_TARGET_DIR)/boot/grub/menu.lst
> +endef
> else
> +define ROOTFS_ISO9660_INITRD
> $(INSTALL) -D -m 0644 $(BINARIES_DIR)/rootfs.cpio$(ROOTFS_CPIO_COMPRESS_EXT) \
> $(ISO9660_TARGET_DIR)/initrd
> +endef
> endif
> - # Use fakeroot to pretend all target binaries are owned by root
> - rm -f $(FAKEROOT_SCRIPT)
> - echo "chown -h -R 0:0 $(ISO9660_TARGET_DIR)" >> $(FAKEROOT_SCRIPT)
> - # Use fakeroot so mkisofs believes the previous fakery
> - echo "$(HOST_DIR)/usr/bin/genisoimage -R -b boot/grub/stage2_eltorito -no-emul-boot " \
> - "-boot-load-size 4 -boot-info-table -o $@ $(ISO9660_TARGET_DIR)" \
> - >> $(FAKEROOT_SCRIPT)
> - chmod a+x $(FAKEROOT_SCRIPT)
> - $(HOST_DIR)/usr/bin/fakeroot -- $(FAKEROOT_SCRIPT)
> - - at rm -f $(FAKEROOT_SCRIPT)
> - - at rm -rf $(ISO9660_TARGET_DIR)
>
> -rootfs-iso9660: $(BINARIES_DIR)/rootfs.iso9660
> +define ROOTFS_ISO9660_PREPARATION
> + $(INSTALL) -D -m 0644 $(GRUB_DIR)/stage2/stage2_eltorito \
> + $(ISO9660_TARGET_DIR)/boot/grub/stage2_eltorito
> + $(INSTALL) -D -m 0644 $(ISO9660_BOOT_MENU) \
> + $(ISO9660_TARGET_DIR)/boot/grub/menu.lst
> + $(INSTALL) -D -m 0644 $(LINUX_IMAGE_PATH) $(ISO9660_TARGET_DIR)/kernel
> + $(ROOTFS_ISO9660_SPLASHSCREEN)
> + $(ROOTFS_ISO9660_INITRD)
> +endef
>
> -rootfs-iso9660-show-depends:
> - @echo $(ROOTFS_ISO9660_DEPENDENCIES)
> +ROOTFS_ISO9660_PRE_GEN_HOOKS += ROOTFS_ISO9660_PREPARATION
>
> -.PHONY: rootfs-iso9660 rootfs-iso9660-show-depends
> +define ROOTFS_ISO9660_CMD
> + $(HOST_DIR)/usr/bin/genisoimage -R -b boot/grub/stage2_eltorito \
> + -no-emul-boot -boot-load-size 4 -boot-info-table \
> + -o $@ $(ISO9660_TARGET_DIR)
Why the fakeroot trickery on genisoimage call is gone? or why is it
not needed anymore?
> +endef
>
> -################################################################################
> -#
> -# Toplevel Makefile options
> -#
> -################################################################################
> -ifeq ($(BR2_TARGET_ROOTFS_ISO9660),y)
> -TARGETS_ROOTFS += rootfs-iso9660
> -endif
> +define ROOTFS_ISO9660_REMOVE_TEMPDIR
> + $(RM) -rf $(ISO9660_TARGET_DIR)
> +endef
> +
> +ROOTFS_ISO9660_POST_GEN_HOOKS += ROOTFS_ISO9660_REMOVE_TEMPDIR
> +
> +$(eval $(call ROOTFS_TARGET,iso9660))
> --
> 2.1.0
>
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
Regards,
--
Samuel
More information about the buildroot
mailing list