On Mon, Sep 6, 2010 at 12:41 AM, Frans Meulenbroeks < [email protected]> wrote:
> 2010/9/6 Khem Raj <[email protected]>: > > On (19/08/10 21:14), Frans Meulenbroeks wrote: > >> 2010/8/19 Chris Larson <[email protected]>: > >> > On Thu, Aug 19, 2010 at 4:29 AM, Frans Meulenbroeks < > >> > [email protected]> 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). > 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 :) -- Christopher Larson clarson at kergoth dot com Founder - BitBake, OpenEmbedded, OpenZaurus Maintainer - Tslib Senior Software Engineer, Mentor Graphics _______________________________________________ Openembedded-devel mailing list [email protected] http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
