diff mbox series

toolchain-scripts: compatibility with unbound variable protection

Message ID 20230104122626.3472721-1-jan.kircher@leica-microsystems.com
State New
Headers show
Series toolchain-scripts: compatibility with unbound variable protection | expand

Commit Message

Jan Kircher Jan. 4, 2023, 12:26 p.m. UTC
Fixed an error when Bash's unbound variable protection is enabled (set -u) and variable LD_LIBRARY_PATH does not exist.

Signed-off-by: Jan Kircher <jan.kircher@leica-microsystems.com>
---
 meta/classes-recipe/toolchain-scripts.bbclass | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

--
2.38.1

Please be advised that this email may contain confidential information. If you are not the intended recipient, please notify us by email by replying to the sender and delete this message. The sender disclaims that the content of this email constitutes an offer to enter into, or the acceptance of, any agreement; provided that the foregoing does not invalidate the binding effect of any digital or other electronic reproduction of a manual signature that is included in any attachment.

Comments

Jan Kircher Jan. 4, 2023, 12:33 p.m. UTC | #1
This also applies to langdale and kirkstone. Kirkstone has this recipe in meta/classes/toolchain-scripts.bbclass.
Alexandre Belloni Jan. 6, 2023, 12:33 p.m. UTC | #2
Hello,

Can you set your git from address so git adds a From: line to your
patch?

git config --global sendemail.from "jan.kircher@leica-microsystems.com"

Should be working fine.

On 04/01/2023 13:26:26+0100, Jan Kircher via lists.openembedded.org wrote:
> Fixed an error when Bash's unbound variable protection is enabled (set -u) and variable LD_LIBRARY_PATH does not exist.
> 
> Signed-off-by: Jan Kircher <jan.kircher@leica-microsystems.com>
> ---
>  meta/classes-recipe/toolchain-scripts.bbclass | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/meta/classes-recipe/toolchain-scripts.bbclass b/meta/classes-recipe/toolchain-scripts.bbclass
> index 3cc823fe63..fa658a6596 100644
> --- a/meta/classes-recipe/toolchain-scripts.bbclass
> +++ b/meta/classes-recipe/toolchain-scripts.bbclass
> @@ -37,7 +37,7 @@ toolchain_create_sdk_env_script () {
>         echo '# http://tldp.org/HOWTO/Program-Library-HOWTO/shared-libraries.html#AEN80' >> $script
>         echo '# http://xahlee.info/UnixResource_dir/_/ldpath.html' >> $script
>         echo '# Only disable this check if you are absolutely know what you are doing!' >> $script
> -       echo 'if [ ! -z "$LD_LIBRARY_PATH" ]; then' >> $script
> +       echo 'if [ ! -z "${LD_LIBRARY_PATH:-}" ]; then' >> $script

Your smtp server is mangling the patch (space vs tab) and it is not
possible to apply it.

>         echo "    echo \"Your environment is misconfigured, you probably need to 'unset LD_LIBRARY_PATH'\"" >> $script
>         echo "    echo \"but please check why this was set in the first place and that it's safe to unset.\"" >> $script
>         echo '    echo "The SDK will not operate correctly in most cases when LD_LIBRARY_PATH is set."' >> $script
> --
> 2.38.1
> 
> Please be advised that this email may contain confidential information. If you are not the intended recipient, please notify us by email by replying to the sender and delete this message. The sender disclaims that the content of this email constitutes an offer to enter into, or the acceptance of, any agreement; provided that the foregoing does not invalidate the binding effect of any digital or other electronic reproduction of a manual signature that is included in any attachment.

This is probably the cause of the issue.

> 
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#175473): https://lists.openembedded.org/g/openembedded-core/message/175473
> Mute This Topic: https://lists.openembedded.org/mt/96048942/3617179
> Group Owner: openembedded-core+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [alexandre.belloni@bootlin.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
Jan Kircher Jan. 11, 2023, 8:54 a.m. UTC | #3
I resubmitted this patch from a different account: https://lists.openembedded.org/g/openembedded-core/topic/96180133
diff mbox series

Patch

diff --git a/meta/classes-recipe/toolchain-scripts.bbclass b/meta/classes-recipe/toolchain-scripts.bbclass
index 3cc823fe63..fa658a6596 100644
--- a/meta/classes-recipe/toolchain-scripts.bbclass
+++ b/meta/classes-recipe/toolchain-scripts.bbclass
@@ -37,7 +37,7 @@  toolchain_create_sdk_env_script () {
        echo '# http://tldp.org/HOWTO/Program-Library-HOWTO/shared-libraries.html#AEN80' >> $script
        echo '# http://xahlee.info/UnixResource_dir/_/ldpath.html' >> $script
        echo '# Only disable this check if you are absolutely know what you are doing!' >> $script
-       echo 'if [ ! -z "$LD_LIBRARY_PATH" ]; then' >> $script
+       echo 'if [ ! -z "${LD_LIBRARY_PATH:-}" ]; then' >> $script
        echo "    echo \"Your environment is misconfigured, you probably need to 'unset LD_LIBRARY_PATH'\"" >> $script
        echo "    echo \"but please check why this was set in the first place and that it's safe to unset.\"" >> $script
        echo '    echo "The SDK will not operate correctly in most cases when LD_LIBRARY_PATH is set."' >> $script