All of lore.kernel.org
 help / color / mirror / Atom feed
From: Khem Raj <raj.khem@gmail.com>
To: openembedded-devel@lists.openembedded.org
Subject: Re: virtclass_install prepend
Date: Sun, 5 Sep 2010 20:50:02 -0700	[thread overview]
Message-ID: <20100906035002.GB5320@gmail.com> (raw)
In-Reply-To: <AANLkTikSBQOi6aC0W8QzfEOk6skMcBmbaJdmE4K+Auwd@mail.gmail.com>

On (19/08/10 21:14), Frans Meulenbroeks wrote:
> 2010/8/19 Chris Larson <clarson@kergoth.com>:
> > On Thu, Aug 19, 2010 at 4:29 AM, Frans Meulenbroeks <
> > fransmeulenbroeks@gmail.com> wrote:
> >
> >> Hi,
> >>
> >> Found this in our reicpes:
> >>
> >> dbus/dbus-glib_0.86.bb:do_install_virtclass-native_prepend() {
> >>
> >
> > This prepends to do_install_virtclass-native, which will then replace
> > do_install when the overrides are applied.  This is prepending to a variable
> > which overrides, as opposed to the other, which is a conditional prepend.
> >  The two are quite different.  It's unlikely that this is what they
> > intended.  Far more likely that it should be changed to
> > _prepend_virtclass-native.
> 
> I have no idea what is intended and I have no dbus knowledge, so can't fix this.
> I'll leave it to someone else to resolve this.

I think the correct fix is as Chris suggested. Please go ahead and fix it.

> 
> Frans
> 
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel



  reply	other threads:[~2010-09-06  3:51 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-08-19 11:29 virtclass_install prepend Frans Meulenbroeks
2010-08-19 16:15 ` Chris Larson
2010-08-19 19:14   ` Frans Meulenbroeks
2010-09-06  3:50     ` Khem Raj [this message]
2010-09-06  7:41       ` Frans Meulenbroeks
2010-09-06 13:54         ` Chris Larson
2010-09-06 16:25           ` Khem Raj
2010-09-06 16:35           ` Frans Meulenbroeks
2010-09-06 20:11             ` Henning Heinold
2010-09-07 13:57               ` Richard Purdie
2010-09-08  0:24                 ` Khem Raj

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=20100906035002.GB5320@gmail.com \
    --to=raj.khem@gmail.com \
    --cc=openembedded-devel@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 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.