From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga14.intel.com ([143.182.124.37]) by linuxtogo.org with esmtp (Exim 4.72) (envelope-from ) id 1SkITV-0002Ic-2m for openembedded-core@lists.openembedded.org; Thu, 28 Jun 2012 19:25:49 +0200 Received: from azsmga002.ch.intel.com ([10.2.17.35]) by azsmga102.ch.intel.com with ESMTP; 28 Jun 2012 10:14:53 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.71,315,1320652800"; d="scan'208";a="117024212" Received: from unknown (HELO [10.255.12.182]) ([10.255.12.182]) by AZSMGA002.ch.intel.com with ESMTP; 28 Jun 2012 10:14:53 -0700 Message-ID: <4FEC910D.9020305@linux.intel.com> Date: Thu, 28 Jun 2012 10:14:53 -0700 From: Saul Wold User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:12.0) Gecko/20120430 Thunderbird/12.0.1 MIME-Version: 1.0 To: Patches and discussions about the oe-core layer References: <1340705423-18430-1-git-send-email-ross.burton@intel.com> In-Reply-To: <1340705423-18430-1-git-send-email-ross.burton@intel.com> Subject: Re: [PATCH] gconf.bbclass: don't register schemas in the install stage X-BeenThere: openembedded-core@lists.openembedded.org X-Mailman-Version: 2.1.11 Precedence: list Reply-To: Patches and discussions about the oe-core layer List-Id: Patches and discussions about the oe-core layer List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 28 Jun 2012 17:25:49 -0000 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit On 06/26/2012 03:10 AM, Ross Burton wrote: > Previously this was installing schemas in the sysroot, which is wrong for native > packages as nothing should touch the sysroot directly, and even more wrong for > non-native packages as the sysroot is irrelevant. > > So, export the environment variable that stops the registration happening at > install time. The postinst script will handle the non-native case, and for the > sysroot I've opened #2648. This isn't a massive problem as nothing to my > knowledge actually installs schemas to the sysroot. > > [YOCTO #2245] > > Signed-off-by: Ross Burton > --- > meta/classes/gconf.bbclass | 12 ++++++++++-- > 1 files changed, 10 insertions(+), 2 deletions(-) > > diff --git a/meta/classes/gconf.bbclass b/meta/classes/gconf.bbclass > index a966c26..fb9f701 100644 > --- a/meta/classes/gconf.bbclass > +++ b/meta/classes/gconf.bbclass > @@ -1,10 +1,18 @@ > DEPENDS += "gconf gconf-native" > > -# This is referenced by the gconf m4 macros and would default to the value hardcoded > -# into gconf at compile time otherwise > +# These are for when gconftool is used natively and the prefix isn't necessarily > +# the sysroot. TODO: replicate the postinst logic for -native packages going > +# into sysroot as they won't be running their own install-time schema > +# registration (disabled below) nor the postinst script (as they don't happen). > export GCONF_SCHEMA_INSTALL_SOURCE = "xml:merged:${STAGING_DIR_NATIVE}${sysconfdir}/gconf/gconf.xml.defaults" > export GCONF_BACKEND_DIR = "${STAGING_LIBDIR_NATIVE}/GConf/2" > > +# Disable install-time schema registration as we're a packaging system so this > +# happens in the postinst script, not at install time. Set both the configure > +# script option and the traditional envionment variable just to make sure. > +EXTRA_OECONF += "--disable-schemas-install" > +export GCONF_DISABLE_MAKEFILE_SCHEMA_INSTALL = "1" > + > gconf_postinst() { > if [ "x$D" != "x" ]; then > exit 1 Merged into OE-Core - Still needs to be merged to 1.2.1 Thanks Sau!