[oe] virtclass_install prepend

[Frans Meulenbroeks]

2010/9/6 Khem Raj <raj.khem at gmail.com>:
> On (19/08/10 21:14), Frans Meulenbroeks wrote:
>> 2010/8/19 Chris Larson <clarson at kergoth.com>:
>> > On Thu, Aug 19, 2010 at 4:29 AM, Frans Meulenbroeks <
>> > fransmeulenbroeks at 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.
>

Well I am a little bit hesitating as I am by no means an expert on the
dbus recipe.

The other question that immediately popped up is:
If the code is wrong and no-one noticed, is it actually needed?

The code is:
do_install_virtclass-native_prepend() {
        install -d ${D}${datadir}/dbus
        install -m 0644 dbus-bus-introspect.xml ${D}${datadir}/dbus
}

Actually if the standard do_install needs to be changed, it generally
raises some alarms with me.
Why would we need to install something that the package normally does
not install?
(of course this could be a bug in the package, but then I would prefer
a patch that we also try to push upstream).

Frans.