* [U-Boot] [PATCH] env: Fix operation of 'make environ'
@ 2017-08-29 19:53 Simon Glass
2017-08-29 20:02 ` Joe Hershberger
0 siblings, 1 reply; 8+ messages in thread
From: Simon Glass @ 2017-08-29 19:53 UTC (permalink / raw)
To: u-boot
This was broken by the recent environment refactoring. Specifically:
$ 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
Fix this by updating the Makefile and adjusting the #include filesnames in
two C files.
Fixes: ec74f5f (Makefile: Rename 'env' target to 'environ')
Reported-by: Måns Rullgård <mans@mansr.com>
Signed-off-by: Simon Glass <sjg@chromium.org>
---
Makefile | 2 +-
tools/env/env_attr.c | 2 +-
tools/env/env_flags.c | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/Makefile b/Makefile
index a0f3bfd2a5..dd9d567811 100644
--- a/Makefile
+++ b/Makefile
@@ -1466,7 +1466,7 @@ checkarmreloc: u-boot
fi
environ: scripts_basic
- $(Q)$(MAKE) $(build)=tools/$@
+ $(Q)$(MAKE) $(build)=tools/env
tools-only: scripts_basic $(version_h) $(timestamp_h)
$(Q)$(MAKE) $(build)=tools
diff --git a/tools/env/env_attr.c b/tools/env/env_attr.c
index 502d4c900b..4d8536335c 100644
--- a/tools/env/env_attr.c
+++ b/tools/env/env_attr.c
@@ -1 +1 @@
-#include "../../common/env_attr.c"
+#include "../../env/attr.c"
diff --git a/tools/env/env_flags.c b/tools/env/env_flags.c
index b261cb8e39..71e13e2021 100644
--- a/tools/env/env_flags.c
+++ b/tools/env/env_flags.c
@@ -1 +1 @@
-#include "../../common/env_flags.c"
+#include "../../env/flags.c"
--
2.14.1.342.g6490525c54-goog
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [U-Boot] [PATCH] env: Fix operation of 'make environ'
2017-08-29 19:53 [U-Boot] [PATCH] env: Fix operation of 'make environ' Simon Glass
@ 2017-08-29 20:02 ` Joe Hershberger
2017-08-29 20:16 ` Simon Glass
0 siblings, 1 reply; 8+ messages in thread
From: Joe Hershberger @ 2017-08-29 20:02 UTC (permalink / raw)
To: u-boot
Hi Simon,
On Tue, Aug 29, 2017 at 2:53 PM, Simon Glass <sjg@chromium.org> wrote:
> This was broken by the recent environment refactoring. Specifically:
>
> $ 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
>
> Fix this by updating the Makefile and adjusting the #include filesnames in
> two C files.
>
> Fixes: ec74f5f (Makefile: Rename 'env' target to 'environ')
> Reported-by: Måns Rullgård <mans@mansr.com>
> Signed-off-by: Simon Glass <sjg@chromium.org>
I assume this was an accident. There is nothing different about this
since the last time.
Cheers,
-Joe
^ permalink raw reply [flat|nested] 8+ messages in thread
* [U-Boot] [PATCH] env: Fix operation of 'make environ'
2017-08-29 20:02 ` Joe Hershberger
@ 2017-08-29 20:16 ` Simon Glass
0 siblings, 0 replies; 8+ messages in thread
From: Simon Glass @ 2017-08-29 20:16 UTC (permalink / raw)
To: u-boot
Hi Joe,
On 30 August 2017 at 04:02, Joe Hershberger <joe.hershberger@ni.com> wrote:
> Hi Simon,
>
> On Tue, Aug 29, 2017 at 2:53 PM, Simon Glass <sjg@chromium.org> wrote:
>> This was broken by the recent environment refactoring. Specifically:
>>
>> $ 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
>>
>> Fix this by updating the Makefile and adjusting the #include filesnames in
>> two C files.
>>
>> Fixes: ec74f5f (Makefile: Rename 'env' target to 'environ')
>> Reported-by: Måns Rullgård <mans@mansr.com>
>> Signed-off-by: Simon Glass <sjg@chromium.org>
>
> I assume this was an accident. There is nothing different about this
> since the last time.
Sorry, yes, please ignore.
Regards,
Simon
^ permalink raw reply [flat|nested] 8+ messages in thread
* [U-Boot] [PATCH] env: Fix operation of 'make environ'
2017-08-28 16:08 ` Joe Hershberger
@ 2017-09-05 7:54 ` Simon Glass
0 siblings, 0 replies; 8+ messages in thread
From: Simon Glass @ 2017-09-05 7:54 UTC (permalink / raw)
To: u-boot
Hi,
On 29 August 2017 at 00:08, Joe Hershberger <joe.hershberger@ni.com> wrote:
>
> On Mon, Aug 28, 2017 at 6:13 AM, Tom Rini <trini@konsulko.com> wrote:
> > On Mon, Aug 28, 2017 at 12:01:51PM +0100, M책ns Rullg책rd wrote:
> >> Simon Glass <sjg@chromium.org> writes:
> >>
> >> > This was broken by the recent environment refactoring. Specifically:
> >> >
> >> > $ 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
> >> >
> >> > Fix this by updating the Makefile and adjusting the #include filesnames in
> >> > two C files.
> >> >
> >> > Fixes: ec74f5f (Makefile: Rename 'env' target to 'environ')
> >> > Reported-by: M책ns Rullg책rd <mans@mansr.com>
> >> > Signed-off-by: Simon Glass <sjg@chromium.org>
> >>
> >> This still breaks any build scripts using the old target (yes, there are
> >> many). That's probably acceptable, but a mention of it in some release
> >> note would be nice.
> >>
> >> Come to think of it, if we're anyway going to break things, how about
> >> giving that target a more descriptive name than environ? I suggest
> >> envtools.
> >
> > Too early, no coffee. Can't we do some make logic so that 'env' is
> > still a target and thus we don't break those scripts? Otherwise, yes,
> > I'll make sure to indicate in at least the v2017.09 release email that
> > we renamed 'env' to envtools or something. Thanks!
>
> It's really just building the fw_printenv tool (which uses a symlink
> for fw_setenv). It seems like a much clearer make target, and source
> dir for that matter, would be fw_printenv.
>
> -Joe
I decided to go with envtools for v2 since even though there is a
symlink, there are two tools intended from this.
If you don't like it let me know and I can change it.
I could not figure out how to make the 'env' target build what we have
now. We make each subdir a target in the main Makefile, so it just
produced a duplicate target.
Regards,
Simon
^ permalink raw reply [flat|nested] 8+ messages in thread
* [U-Boot] [PATCH] env: Fix operation of 'make environ'
2017-08-28 11:13 ` Tom Rini
@ 2017-08-28 16:08 ` Joe Hershberger
2017-09-05 7:54 ` Simon Glass
0 siblings, 1 reply; 8+ messages in thread
From: Joe Hershberger @ 2017-08-28 16:08 UTC (permalink / raw)
To: u-boot
On Mon, Aug 28, 2017 at 6:13 AM, Tom Rini <trini@konsulko.com> wrote:
> On Mon, Aug 28, 2017 at 12:01:51PM +0100, M책ns Rullg책rd wrote:
>> Simon Glass <sjg@chromium.org> writes:
>>
>> > This was broken by the recent environment refactoring. Specifically:
>> >
>> > $ 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
>> >
>> > Fix this by updating the Makefile and adjusting the #include filesnames in
>> > two C files.
>> >
>> > Fixes: ec74f5f (Makefile: Rename 'env' target to 'environ')
>> > Reported-by: M책ns Rullg책rd <mans@mansr.com>
>> > Signed-off-by: Simon Glass <sjg@chromium.org>
>>
>> This still breaks any build scripts using the old target (yes, there are
>> many). That's probably acceptable, but a mention of it in some release
>> note would be nice.
>>
>> Come to think of it, if we're anyway going to break things, how about
>> giving that target a more descriptive name than environ? I suggest
>> envtools.
>
> Too early, no coffee. Can't we do some make logic so that 'env' is
> still a target and thus we don't break those scripts? Otherwise, yes,
> I'll make sure to indicate in at least the v2017.09 release email that
> we renamed 'env' to envtools or something. Thanks!
It's really just building the fw_printenv tool (which uses a symlink
for fw_setenv). It seems like a much clearer make target, and source
dir for that matter, would be fw_printenv.
-Joe
^ permalink raw reply [flat|nested] 8+ messages in thread
* [U-Boot] [PATCH] env: Fix operation of 'make environ'
2017-08-28 11:01 ` Måns Rullgård
@ 2017-08-28 11:13 ` Tom Rini
2017-08-28 16:08 ` Joe Hershberger
0 siblings, 1 reply; 8+ messages in thread
From: Tom Rini @ 2017-08-28 11:13 UTC (permalink / raw)
To: u-boot
On Mon, Aug 28, 2017 at 12:01:51PM +0100, M책ns Rullg책rd wrote:
> Simon Glass <sjg@chromium.org> writes:
>
> > This was broken by the recent environment refactoring. Specifically:
> >
> > $ 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
> >
> > Fix this by updating the Makefile and adjusting the #include filesnames in
> > two C files.
> >
> > Fixes: ec74f5f (Makefile: Rename 'env' target to 'environ')
> > Reported-by: M책ns Rullg책rd <mans@mansr.com>
> > Signed-off-by: Simon Glass <sjg@chromium.org>
>
> This still breaks any build scripts using the old target (yes, there are
> many). That's probably acceptable, but a mention of it in some release
> note would be nice.
>
> Come to think of it, if we're anyway going to break things, how about
> giving that target a more descriptive name than environ? I suggest
> envtools.
Too early, no coffee. Can't we do some make logic so that 'env' is
still a target and thus we don't break those scripts? Otherwise, yes,
I'll make sure to indicate in at least the v2017.09 release email that
we renamed 'env' to envtools or something. Thanks!
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170828/22ddbc2a/attachment.sig>
^ permalink raw reply [flat|nested] 8+ messages in thread
* [U-Boot] [PATCH] env: Fix operation of 'make environ'
2017-08-27 16:48 Simon Glass
@ 2017-08-28 11:01 ` Måns Rullgård
2017-08-28 11:13 ` Tom Rini
0 siblings, 1 reply; 8+ messages in thread
From: Måns Rullgård @ 2017-08-28 11:01 UTC (permalink / raw)
To: u-boot
Simon Glass <sjg@chromium.org> writes:
> This was broken by the recent environment refactoring. Specifically:
>
> $ 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
>
> Fix this by updating the Makefile and adjusting the #include filesnames in
> two C files.
>
> Fixes: ec74f5f (Makefile: Rename 'env' target to 'environ')
> Reported-by: M책ns Rullg책rd <mans@mansr.com>
> Signed-off-by: Simon Glass <sjg@chromium.org>
> ---
>
> Makefile | 2 +-
> tools/env/env_attr.c | 2 +-
> tools/env/env_flags.c | 2 +-
> 3 files changed, 3 insertions(+), 3 deletions(-)
This still breaks any build scripts using the old target (yes, there are
many). That's probably acceptable, but a mention of it in some release
note would be nice.
Come to think of it, if we're anyway going to break things, how about
giving that target a more descriptive name than environ? I suggest
envtools.
> diff --git a/Makefile b/Makefile
> index a0f3bfd2a52..dd9d5678118 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -1466,7 +1466,7 @@ checkarmreloc: u-boot
> fi
>
> environ: scripts_basic
> - $(Q)$(MAKE) $(build)=tools/$@
> + $(Q)$(MAKE) $(build)=tools/env
>
> tools-only: scripts_basic $(version_h) $(timestamp_h)
> $(Q)$(MAKE) $(build)=tools
> diff --git a/tools/env/env_attr.c b/tools/env/env_attr.c
> index 502d4c900bf..4d8536335c3 100644
> --- a/tools/env/env_attr.c
> +++ b/tools/env/env_attr.c
> @@ -1 +1 @@
> -#include "../../common/env_attr.c"
> +#include "../../env/attr.c"
> diff --git a/tools/env/env_flags.c b/tools/env/env_flags.c
> index b261cb8e390..71e13e2021f 100644
> --- a/tools/env/env_flags.c
> +++ b/tools/env/env_flags.c
> @@ -1 +1 @@
> -#include "../../common/env_flags.c"
> +#include "../../env/flags.c"
> --
> 2.14.1.342.g6490525c54-goog
>
--
M책ns Rullg책rd
^ permalink raw reply [flat|nested] 8+ messages in thread
* [U-Boot] [PATCH] env: Fix operation of 'make environ'
@ 2017-08-27 16:48 Simon Glass
2017-08-28 11:01 ` Måns Rullgård
0 siblings, 1 reply; 8+ messages in thread
From: Simon Glass @ 2017-08-27 16:48 UTC (permalink / raw)
To: u-boot
This was broken by the recent environment refactoring. Specifically:
$ 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
Fix this by updating the Makefile and adjusting the #include filesnames in
two C files.
Fixes: ec74f5f (Makefile: Rename 'env' target to 'environ')
Reported-by: Måns Rullgård <mans@mansr.com>
Signed-off-by: Simon Glass <sjg@chromium.org>
---
Makefile | 2 +-
tools/env/env_attr.c | 2 +-
tools/env/env_flags.c | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/Makefile b/Makefile
index a0f3bfd2a52..dd9d5678118 100644
--- a/Makefile
+++ b/Makefile
@@ -1466,7 +1466,7 @@ checkarmreloc: u-boot
fi
environ: scripts_basic
- $(Q)$(MAKE) $(build)=tools/$@
+ $(Q)$(MAKE) $(build)=tools/env
tools-only: scripts_basic $(version_h) $(timestamp_h)
$(Q)$(MAKE) $(build)=tools
diff --git a/tools/env/env_attr.c b/tools/env/env_attr.c
index 502d4c900bf..4d8536335c3 100644
--- a/tools/env/env_attr.c
+++ b/tools/env/env_attr.c
@@ -1 +1 @@
-#include "../../common/env_attr.c"
+#include "../../env/attr.c"
diff --git a/tools/env/env_flags.c b/tools/env/env_flags.c
index b261cb8e390..71e13e2021f 100644
--- a/tools/env/env_flags.c
+++ b/tools/env/env_flags.c
@@ -1 +1 @@
-#include "../../common/env_flags.c"
+#include "../../env/flags.c"
--
2.14.1.342.g6490525c54-goog
^ permalink raw reply related [flat|nested] 8+ messages in thread
end of thread, other threads:[~2017-09-05 7:54 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-08-29 19:53 [U-Boot] [PATCH] env: Fix operation of 'make environ' Simon Glass
2017-08-29 20:02 ` Joe Hershberger
2017-08-29 20:16 ` Simon Glass
-- strict thread matches above, loose matches on Subject: below --
2017-08-27 16:48 Simon Glass
2017-08-28 11:01 ` Måns Rullgård
2017-08-28 11:13 ` Tom Rini
2017-08-28 16:08 ` Joe Hershberger
2017-09-05 7:54 ` Simon Glass
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.