| Message ID | 20161223031508.30532-1-npiggin@gmail.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
El 23/12/16 a las 04:15, Nicholas Piggin escribió: > When using initramfs compression, the data file gets quotes pulled > in from Kconfig, e.g., initramfs_data.cpio".gz" which make does not > match a target and causes rebuild. > > Fix this by filtering out quotes from the Kconfig string. > > While we're here, set the target explicitly so the Makefile does not > have to know about all the suffixes. > > Signed-off-by: Nicholas Piggin <npiggin@gmail.com> > --- > usr/Makefile | 16 ++++++++-------- > 1 file changed, 8 insertions(+), 8 deletions(-) > > diff --git a/usr/Makefile b/usr/Makefile > index 17a513268325..3116b23d44f8 100644 > --- a/usr/Makefile > +++ b/usr/Makefile > @@ -5,8 +5,10 @@ > klibcdirs:; > PHONY += klibcdirs > > -suffix_y = $(CONFIG_INITRAMFS_COMPRESSION) > -AFLAGS_initramfs_data.o += -DINITRAMFS_IMAGE="usr/initramfs_data.cpio$(suffix_y)" > +suffix_y = $(subst $\",,$(CONFIG_INITRAMFS_COMPRESSION)) > +datafile_y = initramfs_data.cpio$(suffix_y) > +AFLAGS_initramfs_data.o += -DINITRAMFS_IMAGE=usr/$(datafile_y) > + > > # Generate builtin.o based on initramfs_data.o > obj-$(CONFIG_BLK_DEV_INITRD) := initramfs_data.o > @@ -14,7 +16,7 @@ obj-$(CONFIG_BLK_DEV_INITRD) := initramfs_data.o > # initramfs_data.o contains the compressed initramfs_data.cpio image. > # The image is included using .incbin, a dependency which is not > # tracked automatically. > -$(obj)/initramfs_data.o: $(obj)/initramfs_data.cpio$(suffix_y) FORCE > +$(obj)/initramfs_data.o: $(obj)/$(datafile_y) FORCE > > ##### > # Generate the initramfs cpio archive > @@ -38,10 +40,8 @@ endif > quiet_cmd_initfs = GEN $@ > cmd_initfs = $(initramfs) -o $@ $(ramfs-args) $(ramfs-input) > > -targets := initramfs_data.cpio.gz initramfs_data.cpio.bz2 \ > - initramfs_data.cpio.lzma initramfs_data.cpio.xz \ > - initramfs_data.cpio.lzo initramfs_data.cpio.lz4 \ > - initramfs_data.cpio > +targets := $(datafile_y) > + > # do not try to update files included in initramfs > $(deps_initramfs): ; > > @@ -51,6 +51,6 @@ $(deps_initramfs): klibcdirs > # 2) There are changes in which files are included (added or deleted) > # 3) If gen_init_cpio are newer than initramfs_data.cpio > # 4) arguments to gen_initramfs.sh changes > -$(obj)/initramfs_data.cpio$(suffix_y): $(obj)/gen_init_cpio $(deps_initramfs) klibcdirs > +$(obj)/$(datafile_y): $(obj)/gen_init_cpio $(deps_initramfs) klibcdirs > $(Q)$(initramfs) -l $(ramfs-input) > $(obj)/.initramfs_data.cpio.d > $(call if_changed,initfs) > -- 2.11.0 Hello! First of all, sorry for taking too long to answer, Christmas kept me awfully busy :( Thanks a lot for catching the issue first, I didn't notice the spurious rebuilds when testing my patches. I have also reviewed and tested this one. I would split the two changes (the quoted string fix and the one to make the Makefile not depend on the extension) into two patches. Other than that feel free to add a Reviewed-by and Tested-by if you need :) Sincerely, Francisco Blas Izquierdo Riera (klondike)
diff --git a/usr/Makefile b/usr/Makefile index 17a513268325..3116b23d44f8 100644 --- a/usr/Makefile +++ b/usr/Makefile @@ -5,8 +5,10 @@ klibcdirs:; PHONY += klibcdirs -suffix_y = $(CONFIG_INITRAMFS_COMPRESSION) -AFLAGS_initramfs_data.o += -DINITRAMFS_IMAGE="usr/initramfs_data.cpio$(suffix_y)" +suffix_y = $(subst $\",,$(CONFIG_INITRAMFS_COMPRESSION)) +datafile_y = initramfs_data.cpio$(suffix_y) +AFLAGS_initramfs_data.o += -DINITRAMFS_IMAGE=usr/$(datafile_y) + # Generate builtin.o based on initramfs_data.o obj-$(CONFIG_BLK_DEV_INITRD) := initramfs_data.o @@ -14,7 +16,7 @@ obj-$(CONFIG_BLK_DEV_INITRD) := initramfs_data.o # initramfs_data.o contains the compressed initramfs_data.cpio image. # The image is included using .incbin, a dependency which is not # tracked automatically. -$(obj)/initramfs_data.o: $(obj)/initramfs_data.cpio$(suffix_y) FORCE +$(obj)/initramfs_data.o: $(obj)/$(datafile_y) FORCE ##### # Generate the initramfs cpio archive @@ -38,10 +40,8 @@ endif quiet_cmd_initfs = GEN $@ cmd_initfs = $(initramfs) -o $@ $(ramfs-args) $(ramfs-input) -targets := initramfs_data.cpio.gz initramfs_data.cpio.bz2 \ - initramfs_data.cpio.lzma initramfs_data.cpio.xz \ - initramfs_data.cpio.lzo initramfs_data.cpio.lz4 \ - initramfs_data.cpio +targets := $(datafile_y) + # do not try to update files included in initramfs $(deps_initramfs): ; @@ -51,6 +51,6 @@ $(deps_initramfs): klibcdirs # 2) There are changes in which files are included (added or deleted) # 3) If gen_init_cpio are newer than initramfs_data.cpio # 4) arguments to gen_initramfs.sh changes -$(obj)/initramfs_data.cpio$(suffix_y): $(obj)/gen_init_cpio $(deps_initramfs) klibcdirs +$(obj)/$(datafile_y): $(obj)/gen_init_cpio $(deps_initramfs) klibcdirs $(Q)$(initramfs) -l $(ramfs-input) > $(obj)/.initramfs_data.cpio.d $(call if_changed,initfs)
When using initramfs compression, the data file gets quotes pulled in from Kconfig, e.g., initramfs_data.cpio".gz" which make does not match a target and causes rebuild. Fix this by filtering out quotes from the Kconfig string. While we're here, set the target explicitly so the Makefile does not have to know about all the suffixes. Signed-off-by: Nicholas Piggin <npiggin@gmail.com> --- usr/Makefile | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-)