Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: Saul Wold <sgw@linux.intel.com>
Cc: openembedded-core@lists.openembedded.org
Subject: Re: [PATCH 2/2] sato-icon-theme: update mapping program location
Date: Tue, 16 Oct 2012 12:37:16 +0100	[thread overview]
Message-ID: <1350387436.2185.39.camel@ted> (raw)
In-Reply-To: <507CDF3E.5070808@linux.intel.com>

On Mon, 2012-10-15 at 21:14 -0700, Saul Wold wrote:
> On 10/15/2012 03:19 PM, Saul Wold wrote:
> > When we changed the /usr/libexec default to be
> > /usr/lib/<pn>, the icon name mapping needed to
> > be updated also.
> >
> > Signed-off-by: Saul Wold <sgw@linux.intel.com>
> > ---
> >   .../sato-icon-theme/sato-icon-theme.inc            |    2 +-
> >   .../sato-icon-theme/sato-icon-theme_0.4.1.bb       |    2 +-
> >   2 files changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/meta/recipes-sato/sato-icon-theme/sato-icon-theme.inc b/meta/recipes-sato/sato-icon-theme/sato-icon-theme.inc
> > index 0bd4814..b07d8c3 100644
> > --- a/meta/recipes-sato/sato-icon-theme/sato-icon-theme.inc
> > +++ b/meta/recipes-sato/sato-icon-theme/sato-icon-theme.inc
> > @@ -12,7 +12,7 @@ inherit autotools pkgconfig allarch gtk-icon-cache
> >
> >   FILES_${PN} += "${datadir}"
> >
> > -EXTRA_OECONF += "--with-iconmap=${STAGING_LIBDIR_NATIVE}/../libexec/icon-name-mapping"
> > +EXTRA_OECONF += "--with-iconmap=${STAGING_LIBEXECDIR_NATIVE}/icon-name-mapping"
> >
> This is still wrong apparently since icon-naming-mapping is not placed 
> in sato-icon-theme, but in /usr/lib/icon-naming-utils.  So my original 
> patch should be correct.

No, it should be:

EXTRA_OECONF += "--with-iconmap=${STAGING_LIBEXECDIR_NATIVE}/../icon-name-mapping"

Cheers,

Richard




      reply	other threads:[~2012-10-16 11:50 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-10-15 22:19 [PATCH 1/2] bitbake.conf: change libexecdir to ${libdir}/${BPN} Saul Wold
2012-10-15 22:19 ` [PATCH 2/2] sato-icon-theme: update mapping program location Saul Wold
2012-10-16  4:14   ` Saul Wold
2012-10-16 11:37     ` Richard Purdie [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=1350387436.2185.39.camel@ted \
    --to=richard.purdie@linuxfoundation.org \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=sgw@linux.intel.com \
    /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