Message ID | 1394098949-31981-3-git-send-email-fabio.porcedda@gmail.com |
---|---|
State | Accepted |
Headers | show |
On 06/03/14 10:42, Fabio Porcedda wrote: > Because now the toolchain dependency is automatically added by the > package infrastructure the BASE_TARGETS variable is useless so just > remove it. > > Signed-off-by: Fabio Porcedda <fabio.porcedda@gmail.com> Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be> Regards, Arnout > --- > Makefile | 10 ++++------ > docs/manual/how-buildroot-works.txt | 9 ++++----- > 2 files changed, 8 insertions(+), 11 deletions(-) > > diff --git a/Makefile b/Makefile > index 6eb9743..665865b 100644 > --- a/Makefile > +++ b/Makefile > @@ -300,9 +300,7 @@ GNU_HOST_NAME:=$(shell support/gnuconfig/config.guess) > # > ################################################################################ > > -BASE_TARGETS = toolchain > - > -TARGETS:= > +TARGETS:= toolchain > > # silent mode requested? > QUIET:=$(if $(findstring s,$(MAKEFLAGS)),-q) > @@ -411,7 +409,7 @@ endif > > include fs/common.mk > > -TARGETS_SOURCE:=$(patsubst %,%-source,$(TARGETS) $(BASE_TARGETS)) > +TARGETS_SOURCE:=$(patsubst %,%-source,$(TARGETS)) > TARGETS_DIRCLEAN:=$(patsubst %,%-dirclean,$(TARGETS)) > > # host-* dependencies have to be handled specially, as those aren't > @@ -433,7 +431,7 @@ HOST_DEPS = $(sort $(foreach dep,\ > HOST_SOURCE += $(addsuffix -source,$(sort $(TARGETS_HOST_DEPS) $(HOST_DEPS))) > > TARGETS_LEGAL_INFO:=$(patsubst %,%-legal-info,\ > - $(TARGETS) $(BASE_TARGETS) $(TARGETS_HOST_DEPS) $(HOST_DEPS)))) > + $(TARGETS) $(TARGETS_HOST_DEPS) $(HOST_DEPS)))) > > dirs: $(BUILD_DIR) $(STAGING_DIR) $(TARGET_DIR) \ > $(HOST_DIR) $(BINARIES_DIR) $(STAMP_DIR) > @@ -452,7 +450,7 @@ world: target-post-image > .PHONY: all world toolchain dirs clean distclean source outputmakefile \ > legal-info legal-info-prepare legal-info-clean printvars \ > target-finalize target-post-image \ > - $(BASE_TARGETS) $(TARGETS) $(TARGETS_ROOTFS) \ > + $(TARGETS) $(TARGETS_ROOTFS) \ > $(TARGETS_DIRCLEAN) $(TARGETS_SOURCE) $(TARGETS_LEGAL_INFO) \ > $(BUILD_DIR) $(STAGING_DIR) $(TARGET_DIR) \ > $(HOST_DIR) $(BINARIES_DIR) $(STAMP_DIR) > diff --git a/docs/manual/how-buildroot-works.txt b/docs/manual/how-buildroot-works.txt > index 68ddd13..a96e45b 100644 > --- a/docs/manual/how-buildroot-works.txt > +++ b/docs/manual/how-buildroot-works.txt > @@ -56,11 +56,10 @@ configuration is done): > +stamps+, etc. in the output directory (+output/+ by default, > another value can be specified using +O=+) > > -* Generate all the targets listed in the +BASE_TARGETS+ variable. When > - an internal toolchain is used, this means generating the > - cross-compilation toolchain. When an external toolchain is used, > - this means checking the features of the external toolchain and > - importing it into the Buildroot environment. > +* Generate the toolchain target. When an internal toolchain is used, this > + means generating the cross-compilation toolchain. When an external > + toolchain is used, this means checking the features of the external > + toolchain and importing it into the Buildroot environment. > > * Generate all the targets listed in the +TARGETS+ variable. This > variable is filled by all the individual components' >
diff --git a/Makefile b/Makefile index 6eb9743..665865b 100644 --- a/Makefile +++ b/Makefile @@ -300,9 +300,7 @@ GNU_HOST_NAME:=$(shell support/gnuconfig/config.guess) # ################################################################################ -BASE_TARGETS = toolchain - -TARGETS:= +TARGETS:= toolchain # silent mode requested? QUIET:=$(if $(findstring s,$(MAKEFLAGS)),-q) @@ -411,7 +409,7 @@ endif include fs/common.mk -TARGETS_SOURCE:=$(patsubst %,%-source,$(TARGETS) $(BASE_TARGETS)) +TARGETS_SOURCE:=$(patsubst %,%-source,$(TARGETS)) TARGETS_DIRCLEAN:=$(patsubst %,%-dirclean,$(TARGETS)) # host-* dependencies have to be handled specially, as those aren't @@ -433,7 +431,7 @@ HOST_DEPS = $(sort $(foreach dep,\ HOST_SOURCE += $(addsuffix -source,$(sort $(TARGETS_HOST_DEPS) $(HOST_DEPS))) TARGETS_LEGAL_INFO:=$(patsubst %,%-legal-info,\ - $(TARGETS) $(BASE_TARGETS) $(TARGETS_HOST_DEPS) $(HOST_DEPS)))) + $(TARGETS) $(TARGETS_HOST_DEPS) $(HOST_DEPS)))) dirs: $(BUILD_DIR) $(STAGING_DIR) $(TARGET_DIR) \ $(HOST_DIR) $(BINARIES_DIR) $(STAMP_DIR) @@ -452,7 +450,7 @@ world: target-post-image .PHONY: all world toolchain dirs clean distclean source outputmakefile \ legal-info legal-info-prepare legal-info-clean printvars \ target-finalize target-post-image \ - $(BASE_TARGETS) $(TARGETS) $(TARGETS_ROOTFS) \ + $(TARGETS) $(TARGETS_ROOTFS) \ $(TARGETS_DIRCLEAN) $(TARGETS_SOURCE) $(TARGETS_LEGAL_INFO) \ $(BUILD_DIR) $(STAGING_DIR) $(TARGET_DIR) \ $(HOST_DIR) $(BINARIES_DIR) $(STAMP_DIR) diff --git a/docs/manual/how-buildroot-works.txt b/docs/manual/how-buildroot-works.txt index 68ddd13..a96e45b 100644 --- a/docs/manual/how-buildroot-works.txt +++ b/docs/manual/how-buildroot-works.txt @@ -56,11 +56,10 @@ configuration is done): +stamps+, etc. in the output directory (+output/+ by default, another value can be specified using +O=+) -* Generate all the targets listed in the +BASE_TARGETS+ variable. When - an internal toolchain is used, this means generating the - cross-compilation toolchain. When an external toolchain is used, - this means checking the features of the external toolchain and - importing it into the Buildroot environment. +* Generate the toolchain target. When an internal toolchain is used, this + means generating the cross-compilation toolchain. When an external + toolchain is used, this means checking the features of the external + toolchain and importing it into the Buildroot environment. * Generate all the targets listed in the +TARGETS+ variable. This variable is filled by all the individual components'
Because now the toolchain dependency is automatically added by the package infrastructure the BASE_TARGETS variable is useless so just remove it. Signed-off-by: Fabio Porcedda <fabio.porcedda@gmail.com> --- Makefile | 10 ++++------ docs/manual/how-buildroot-works.txt | 9 ++++----- 2 files changed, 8 insertions(+), 11 deletions(-)