[oe] virtclass_install prepend

[Khem Raj]

On Tue, Sep 7, 2010 at 6:57 AM, Richard Purdie <rpurdie at rpsys.net> wrote:
> On Mon, 2010-09-06 at 22:11 +0200, Henning Heinold wrote:
>> > >> 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).
>> > >>
>> > >
>> > > Agreed, that is odd.  If it's something that dbus using things might need,
>> > > why is it only prepended for the native version, and as you say, why is it
>> > > not installed by the upstream install target.  I'd do a git blame and figure
>> > > out who added that and ask them :)
>> > > --
>> >
>> > Henning,
>> > Git blame gives your name as the person who created those lines.
>> > Care to comment?
>> >
>> > Frans
>>
>> if I remember correctly this file is generated in the build process. But somehow
>> in the later versions of dbus-glib there is somehow an error so this file gets not installed
>> at all in the native variant. So I did it per hand.
>
> I suspect that code could well have come from Poky. In this case the
> native recipe generates a file which is then used by the target builds.
> It can't be generated in the target recipe as that would mean running
> target binaries.
>
> So it is in fact a perfectly valid thing to do in this case.

wow cool. This needs to be in the comments in that code.

>
> Cheers,
>
> Richard
>
>
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel at lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
>