Hello Richard,

Can you merge this on Dunfell.

Noor
-----Original Message-----
From: openembedded-core@lists.openembedded.org 
[mailto:openembedded-core@lists.openembedded.org] On Behalf Of Hussain, Ahsan
Sent: Monday, June 29, 2020 2:16 PM
To: openembedded-core@lists.openembedded.org
Subject: [OE-core] [dunfell][PATCH 2/2] u-boot: avoid blind merging all *.cfg

From: Jens Rehsack <rehs...@gmail.com>

This fixes u-boot-qoriq breakage of:

    commit 4fde8c8f479f5acd24fb6e0d0b9b4dc94d6f560b
    Author: Ming Liu <ming....@toradex.com>
    Date:   Wed Jun 3 13:56:01 2020 +0200

        u-boot: support merging .cfg files for UBOOT_CONFIG

        U-boot recipe supports .cfg files in SRC_URI, but they would be merged
        to .config during do_configure only when UBOOT_MACHINE is set, we
        should also support merging .cfg files for UBOOT_CONFIG.

The intension of 4fde8c8 looks more than append delta-config snippets to u-boot 
config and should probably be rewritten to express that much better than 
implicitely assume all "*.cfg" are for merging into .config.

Signed-off-by: Jens Rehsack <s...@netbsd.org>
Signed-off-by: Richard Purdie <richard.pur...@linuxfoundation.org>
---
 meta/recipes-bsp/u-boot/u-boot.inc | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/meta/recipes-bsp/u-boot/u-boot.inc 
b/meta/recipes-bsp/u-boot/u-boot.inc
index 8cfd25020c..7d241916eb 100644
--- a/meta/recipes-bsp/u-boot/u-boot.inc
+++ b/meta/recipes-bsp/u-boot/u-boot.inc
@@ -85,8 +85,10 @@ do_configure () {
                 j=$(expr $j + 1);
                 if [ $j -eq $i ]; then
                     oe_runmake -C ${S} O=${B}/${config} ${config}
-                    merge_config.sh -m -O ${B}/${config} 
${B}/${config}/.config ${@" ".join(find_cfgs(d))}
-                    oe_runmake -C ${S} O=${B}/${config} oldconfig
+                    if [ test -n "${@' '.join(find_cfgs(d))}" ]; then
+                        merge_config.sh -m -O ${B}/${config} 
${B}/${config}/.config ${@" ".join(find_cfgs(d))}
+                        oe_runmake -C ${S} O=${B}/${config} oldconfig
+                    fi
                 fi
             done
             unset j
@@ -119,7 +121,7 @@ do_compile () {
                echo ${UBOOT_LOCALVERSION} > ${S}/.scmversion
        fi
 
-    if [ -n "${UBOOT_CONFIG}" ]
+    if [ -n "${UBOOT_CONFIG}" -o -n "${UBOOT_DELTA_CONFIG}" ]
     then
         unset i j k
         for config in ${UBOOT_MACHINE}; do
--
2.25.1

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.

View/Reply Online (#140291): 
https://lists.openembedded.org/g/openembedded-core/message/140291
Mute This Topic: https://lists.openembedded.org/mt/75187818/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub  
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to