Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Alexandre Belloni <alexandre.belloni@bootlin.com>
To: jan.kircher@leica-microsystems.com
Cc: openembedded-core@lists.openembedded.org
Subject: Re: [OE-core] [PATCH] toolchain-scripts: compatibility with unbound variable protection
Date: Fri, 6 Jan 2023 13:33:19 +0100	[thread overview]
Message-ID: <Y7gVD6T3z5fuhX79@mail.local> (raw)
In-Reply-To: <20230104122626.3472721-1-jan.kircher@leica-microsystems.com>

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]
> -=-=-=-=-=-=-=-=-=-=-=-
> 


-- 
Alexandre Belloni, co-owner and COO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com


  parent reply	other threads:[~2023-01-06 12:33 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-01-04 12:26 [PATCH] toolchain-scripts: compatibility with unbound variable protection Jan Kircher
2023-01-04 12:33 ` Jan Kircher
2023-01-06 12:33 ` Alexandre Belloni [this message]
2023-01-11  8:54   ` Jan Kircher

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=Y7gVD6T3z5fuhX79@mail.local \
    --to=alexandre.belloni@bootlin.com \
    --cc=jan.kircher@leica-microsystems.com \
    --cc=openembedded-core@lists.openembedded.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox