On Tue, Sep 7, 2010 at 6:57 AM, Richard Purdie <[email protected]> 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
> [email protected]
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
>

_______________________________________________
Openembedded-devel mailing list
[email protected]
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel

Reply via email to