All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Stiffler, Jacob" <j-stiffler@ti.com>
To: "Dmytriyenko, Denys" <denys@ti.com>
Cc: "meta-arago@arago-project.org" <meta-arago@arago-project.org>
Subject: Re: [master/thud/ti2019.03][PATCH] meta-toolchain-arago-qte: add quotes to vars
Date: Fri, 20 Sep 2019 19:14:44 +0000	[thread overview]
Message-ID: <949f3a6c0a8f461f8265218331d91ef4@ti.com> (raw)
In-Reply-To: <20190920181757.GI5351@beryl>

I am not sure what is require for qmake, but this was the effect with bash. If dash is the shell, then there is an error exporting the variable because of the whitespace.

In summary, this causes no change in behavior if bash is the shell.

> -----Original Message-----
> From: Dmytriyenko, Denys
> Sent: Friday, September 20, 2019 2:18 PM
> To: Stiffler, Jacob
> Cc: meta-arago@arago-project.org
> Subject: Re: [meta-arago] [master/thud/ti2019.03][PATCH] meta-toolchain-
> arago-qte: add quotes to vars
> 
> BTW, do we also need to pass --sysroot parameter for qmake CC vars, like is
> done for main CC vars here:
> http://arago-project.org/git/?p=meta-
> arago.git;a=commitdiff;h=4e87564c39e214a61cbf0088ecf1a3f5a3f01bd5
> 
> 
> On Fri, Sep 20, 2019 at 09:16:05AM -0400, Jacob Stiffler wrote:
> > * Add quotes around values which may contain spaces
> >
> > Signed-off-by: Jacob Stiffler <j-stiffler@ti.com>
> > ---
> >  .../recipes-core/meta/meta-toolchain-arago-qte.bb              | 10 +++++-----
> >  1 file changed, 5 insertions(+), 5 deletions(-)
> >
> > diff --git a/meta-arago-extras/recipes-core/meta/meta-toolchain-arago-
> qte.bb b/meta-arago-extras/recipes-core/meta/meta-toolchain-arago-
> qte.bb
> > index b03b639..d313939 100644
> > --- a/meta-arago-extras/recipes-core/meta/meta-toolchain-arago-qte.bb
> > +++ b/meta-arago-extras/recipes-core/meta/meta-toolchain-arago-qte.bb
> > @@ -5,7 +5,7 @@ TOOLCHAIN_SUFFIX ?= "-qte-sdk"
> >
> >  require meta-toolchain-arago.bb
> >
> > -PR = "r19"
> > +PR = "r20"
> >
> >  # There could be qt5, qt4e and qt4x11 providers, but we don't support
> qt4x11 for now
> >  QT_DIR_NAME = "${@oe.utils.conditional('QT_PROVIDER', 'qt5', '', 'qtopia',
> d)}"
> > @@ -19,10 +19,10 @@ toolchain_create_sdk_env_script_append() {
> >  	echo 'export OE_QMAKE_CFLAGS="$CFLAGS"' >> $script
> >  	echo 'export OE_QMAKE_CXXFLAGS="$CXXFLAGS"' >> $script
> >  	echo 'export OE_QMAKE_LDFLAGS="$LDFLAGS"' >> $script
> > -	echo 'export OE_QMAKE_CC=$CC' >> $script
> > -	echo 'export OE_QMAKE_CXX=$CXX' >> $script
> > -	echo 'export OE_QMAKE_LINK=$CXX' >> $script
> > -	echo 'export OE_QMAKE_AR=$AR' >> $script
> > +	echo 'export OE_QMAKE_CC="$CC"' >> $script
> > +	echo 'export OE_QMAKE_CXX="$CXX"' >> $script
> > +	echo 'export OE_QMAKE_LINK="$CXX"' >> $script
> > +	echo 'export OE_QMAKE_AR="$AR"' >> $script
> >  	echo 'export OE_QMAKE_PREFIX_QT=${prefix}' >> $script
> >  	echo 'export OE_QMAKE_LIBDIR_QT=${libdir}' >> $script
> >  	echo 'export
> OE_QMAKE_INCDIR_QT=${includedir}/${QT_DIR_NAME}' >> $script
> > --
> > 2.7.4
> >
> > _______________________________________________
> > meta-arago mailing list
> > meta-arago@arago-project.org
> > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago


      reply	other threads:[~2019-09-20 19:15 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-09-20 13:16 [master/thud/ti2019.03][PATCH] meta-toolchain-arago-qte: add quotes to vars Jacob Stiffler
2019-09-20 18:17 ` Denys Dmytriyenko
2019-09-20 19:14   ` Stiffler, Jacob [this message]

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=949f3a6c0a8f461f8265218331d91ef4@ti.com \
    --to=j-stiffler@ti.com \
    --cc=denys@ti.com \
    --cc=meta-arago@arago-project.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 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.