All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH] Makefile: unexport 'DEVICE_TREE' environment variable
@ 2022-02-23 12:41 Peter Korsgaard
  2022-02-23 14:05 ` Yann E. MORIN
  0 siblings, 1 reply; 2+ messages in thread
From: Peter Korsgaard @ 2022-02-23 12:41 UTC (permalink / raw)
  To: buildroot

U-Boot looks for the environment variable DEVICE_TREE and uses its value if
set instead of the CONFIG_DEFAULT_DEVICE_TREE configuration option since
v2021.01, more specifically commit c0f1ebe9c1b9745e (binman: Allow selecting
default FIT configuration) - So unexport it like we do for other
"troublesome" environment variables to ensure consistent behaviour.

Reported-by: Neal Frager <nealf@xilinx.com>
Signed-off-by: Peter Korsgaard <peter@korsgaard.com>
---
 Makefile | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Makefile b/Makefile
index d92438166f..0ed93e600d 100644
--- a/Makefile
+++ b/Makefile
@@ -426,6 +426,7 @@ unexport O
 unexport GCC_COLORS
 unexport PLATFORM
 unexport OS
+unexport DEVICE_TREE
 
 GNU_HOST_NAME := $(shell support/gnuconfig/config.guess)
 
-- 
2.30.2

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [Buildroot] [PATCH] Makefile: unexport 'DEVICE_TREE' environment variable
  2022-02-23 12:41 [Buildroot] [PATCH] Makefile: unexport 'DEVICE_TREE' environment variable Peter Korsgaard
@ 2022-02-23 14:05 ` Yann E. MORIN
  0 siblings, 0 replies; 2+ messages in thread
From: Yann E. MORIN @ 2022-02-23 14:05 UTC (permalink / raw)
  To: Peter Korsgaard; +Cc: buildroot

Peter, All,

On 2022-02-23 13:41 +0100, Peter Korsgaard spake thusly:
> U-Boot looks for the environment variable DEVICE_TREE and uses its value if
> set instead of the CONFIG_DEFAULT_DEVICE_TREE configuration option since
> v2021.01, more specifically commit c0f1ebe9c1b9745e (binman: Allow selecting
> default FIT configuration) - So unexport it like we do for other
> "troublesome" environment variables to ensure consistent behaviour.
> 
> Reported-by: Neal Frager <nealf@xilinx.com>
> Signed-off-by: Peter Korsgaard <peter@korsgaard.com>

Applied to master, thanks.

Regards,
Yann E. MORIN.

> ---
>  Makefile | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/Makefile b/Makefile
> index d92438166f..0ed93e600d 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -426,6 +426,7 @@ unexport O
>  unexport GCC_COLORS
>  unexport PLATFORM
>  unexport OS
> +unexport DEVICE_TREE
>  
>  GNU_HOST_NAME := $(shell support/gnuconfig/config.guess)
>  
> -- 
> 2.30.2
> 
> _______________________________________________
> buildroot mailing list
> buildroot@buildroot.org
> https://lists.buildroot.org/mailman/listinfo/buildroot

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 561 099 427 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2022-02-23 14:05 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-02-23 12:41 [Buildroot] [PATCH] Makefile: unexport 'DEVICE_TREE' environment variable Peter Korsgaard
2022-02-23 14:05 ` Yann E. MORIN

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.