[oe] [PATCH] gnome.bbclass: remove unneeded scrollkeeper stuff from gnome packages.
Phil Blundell
philb at gnu.org
Sat May 30 23:42:41 UTC 2009
On Sun, 2009-05-31 at 00:06 +0100, Phil Blundell wrote:
> On Sat, 2009-05-30 at 13:53 -0700, Tom Rini wrote:
> > On Mon, May 25, 2009 at 01:17:53PM +0100, Phil Blundell wrote:
> > > On Mon, 2009-05-25 at 14:18 +0300, Ihar Hrachyshka wrote:
> > > > Scrollkeeper generates its XML database for every package when
> > > > installing it. The problem is that while building it sees only specific
> > > > package scrollkeeper stuff so we get wrong scrollkeeper xml file
> > > > generated for every gnome.bbclass package. Including the file into every
> > > > gnome package results in package contents conflicts.
> > > >
> > > > -PACKAGES =+ "${PN}-scrollkeeper-leftovers"
> > > > -FILES_${PN}-scrollkeeper-leftovers = "${localstatedir}/lib/scrollkeeper"
> > > > -
> > > > gnome_stage_includes() {
> > > > autotools_stage_includes
> > > > }
> > > > +
> > > > +do_install_append() {
> > > > + rm -rf ${D}${localstatedir}/lib/scrollkeeper/*
> > > > +}
> > > > +
> > >
> > > That change looks pretty reasonable to me, though maybe you want to
> > > delete ${localstatedir}/lib/scrollkeeper itself (and its parents, if
> > > they're empty) as well as the contents of the directory.
> > >
> > > In the longer term I guess we should figure out a general-purpose
> > > solution to all this scrollkeeper stuff. I see that some packages are
> > > configuring themselves --disable-scrollkeeper, others seem to be
> > > patching the scrollkeeper bits out of their configure scripts and yet
> > > others (notably empathy) have, effectively, a local copy of the bits
> > > you've just removed from gnome.bbclass.
> >
> > Do you want more changes still? I don't see a SOB line, so... Thanks.
>
> I think it's fine to go in as it is. We can always enhance it later.
Per Tom's request :-)
Acked-by: Phil Blundell <philb at gnu.org>
p.
More information about the Openembedded-devel
mailing list