Message ID | 20170803182217.149285-2-sjg@chromium.org |
---|---|
State | Accepted |
Delegated to: | Tom Rini |
Headers | show |
On Thu, Aug 03, 2017 at 12:21:48PM -0600, Simon Glass wrote: > This target stops us using 'env' as a subdirectory. It is not mentioned in > the help so seems to be an internal target. Rename it. > > Signed-off-by: Simon Glass <sjg@chromium.org> > Reviewed-by: Tom Rini <trini@konsulko.com> Applied to u-boot/master, thanks!
Simon Glass <sjg@chromium.org> writes: > This target stops us using 'env' as a subdirectory. It is not mentioned in > the help so seems to be an internal target. Rename it. > > Signed-off-by: Simon Glass <sjg@chromium.org> > Reviewed-by: Tom Rini <trini@konsulko.com> > --- > > Changes in v5: None > Changes in v4: None > Changes in v3: None > Changes in v2: None > > Makefile | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/Makefile b/Makefile > index 50a002e72f..04a22a6818 100644 > --- a/Makefile > +++ b/Makefile > @@ -1463,14 +1463,14 @@ checkarmreloc: u-boot > false; \ > fi > > -env: scripts_basic > +environ: scripts_basic > $(Q)$(MAKE) $(build)=tools/$@ This broke things: $ make environ scripts/Makefile.build:59: tools/environ/Makefile: No such file or directory make[1]: *** No rule to make target 'tools/environ/Makefile'. Stop. make: *** [Makefile:1469: environ] Error 2 The 'env' target was used to build the fw_{print,set}env tools in the tools/env directory. The make rule relies on the target having the same name as the subdirectory. > tools-only: scripts_basic $(version_h) $(timestamp_h) > $(Q)$(MAKE) $(build)=tools > > tools-all: export HOST_TOOLS_ALL=y > -tools-all: env tools ; > +tools-all: environ tools ; > > cross_tools: export CROSS_BUILD_TOOLS=y > cross_tools: tools ; > -- > 2.14.0.rc1.383.gd1ce394fe2-goog >
Hi Mans, On 18 August 2017 at 08:31, Måns Rullgård <mans@mansr.com> wrote: > Simon Glass <sjg@chromium.org> writes: > >> This target stops us using 'env' as a subdirectory. It is not mentioned in >> the help so seems to be an internal target. Rename it. >> >> Signed-off-by: Simon Glass <sjg@chromium.org> >> Reviewed-by: Tom Rini <trini@konsulko.com> >> --- >> >> Changes in v5: None >> Changes in v4: None >> Changes in v3: None >> Changes in v2: None >> >> Makefile | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/Makefile b/Makefile >> index 50a002e72f..04a22a6818 100644 >> --- a/Makefile >> +++ b/Makefile >> @@ -1463,14 +1463,14 @@ checkarmreloc: u-boot >> false; \ >> fi >> >> -env: scripts_basic >> +environ: scripts_basic >> $(Q)$(MAKE) $(build)=tools/$@ > > This broke things: > > $ make environ > scripts/Makefile.build:59: tools/environ/Makefile: No such file or directory > make[1]: *** No rule to make target 'tools/environ/Makefile'. Stop. > make: *** [Makefile:1469: environ] Error 2 > > The 'env' target was used to build the fw_{print,set}env tools in the > tools/env directory. The make rule relies on the target having the same > name as the subdirectory. > >> tools-only: scripts_basic $(version_h) $(timestamp_h) >> $(Q)$(MAKE) $(build)=tools >> >> tools-all: export HOST_TOOLS_ALL=y >> -tools-all: env tools ; >> +tools-all: environ tools ; >> >> cross_tools: export CROSS_BUILD_TOOLS=y >> cross_tools: tools ; >> -- >> 2.14.0.rc1.383.gd1ce394fe2-goog >> > > -- > Måns Rullgård Thanks for the report. I sent this patch: http://patchwork.ozlabs.org/patch/806277/ Regards, Simon
diff --git a/Makefile b/Makefile index 50a002e72f..04a22a6818 100644 --- a/Makefile +++ b/Makefile @@ -1463,14 +1463,14 @@ checkarmreloc: u-boot false; \ fi -env: scripts_basic +environ: scripts_basic $(Q)$(MAKE) $(build)=tools/$@ tools-only: scripts_basic $(version_h) $(timestamp_h) $(Q)$(MAKE) $(build)=tools tools-all: export HOST_TOOLS_ALL=y -tools-all: env tools ; +tools-all: environ tools ; cross_tools: export CROSS_BUILD_TOOLS=y cross_tools: tools ;