[PATCH 0/2] fix unrecognised options

[PATCH 0/2] fix unrecognised options

[Robert Yang]

The following changes since commit e3e1d52ca34a975739e79e4baeb28cf04e5e49c2:

  bitbake/lib/bb/cooker.py: remove a duplicated self.data (2014-01-04 01:22:32 +0800)

are available in the git repository at:

  git://git.pokylinux.org/poky-contrib robert/warning
  http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=robert/warning

Robert Yang (2):
  gconf.bbclass: remove --disable-schemas-install
  mime.bbclass: remove the --disable-update-mimedb

 meta/classes/gconf.bbclass | 6 ------
 meta/classes/mime.bbclass  | 2 --
 2 files changed, 8 deletions(-)

-- 
1.8.3.1

[PATCH 1/2] gconf.bbclass: remove --disable-schemas-install

[Robert Yang]

Gnome has moved the schemas to the gsettings-desktop-schemas pkg, so
there is no --disable-schemas-install option any more, and we already
have the gsettings-desktop-schemas recipe, so remove it, it wil fix
the warning:

configure was passed unrecognised options: --disable-schemas-install

Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
---
 meta/classes/gconf.bbclass | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/meta/classes/gconf.bbclass b/meta/classes/gconf.bbclass
index e9076b2..8ebb512 100644
--- a/meta/classes/gconf.bbclass
+++ b/meta/classes/gconf.bbclass
@@ -7,12 +7,6 @@ DEPENDS += "gconf gconf-native"
 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
 	export GCONF_CONFIG_SOURCE="xml::$D${sysconfdir}/gconf/gconf.xml.defaults"
-- 
1.8.3.1

Re: [PATCH 1/2] gconf.bbclass: remove --disable-schemas-install

[Burton, Ross]

On 3 January 2014 17:27, Robert Yang <liezhi.yang@windriver.com> wrote:
> Gnome has moved the schemas to the gsettings-desktop-schemas pkg, so
> there is no --disable-schemas-install option any more, and we already
> have the gsettings-desktop-schemas recipe, so remove it, it wil fix
> the warning:
>
> configure was passed unrecognised options: --disable-schemas-install

I'd NAK this but it's too late as I see it's been merged.

*Some* packages have migrated away from gconf to gsettings.  Not all
packages have done this, so anything still using gconf is now broken.
Note that almost nothing in oe-core uses gsettings as we're still
using the GTK+ 2 stack generally.

The problem is that people are using gnome.bbclass which is a kitchen
sink class, inheriting many other classes that may or may not be used.
 To resolve this warning gnome.bbclass should extend the sanity check
whitelist with --disable-schemas-install as it is the class causing
gconf.bbclass to be inherited on recipes that don't use gconf.

Ross

Re: [PATCH 1/2] gconf.bbclass: remove --disable-schemas-install

[Robert Yang]

On 01/06/2014 11:47 PM, Burton, Ross wrote:
> On 3 January 2014 17:27, Robert Yang <liezhi.yang@windriver.com> wrote:
>> Gnome has moved the schemas to the gsettings-desktop-schemas pkg, so
>> there is no --disable-schemas-install option any more, and we already
>> have the gsettings-desktop-schemas recipe, so remove it, it wil fix
>> the warning:
>>
>> configure was passed unrecognised options: --disable-schemas-install
>
> I'd NAK this but it's too late as I see it's been merged.
>
> *Some* packages have migrated away from gconf to gsettings.  Not all
> packages have done this, so anything still using gconf is now broken.
> Note that almost nothing in oe-core uses gsettings as we're still
> using the GTK+ 2 stack generally.
>

Hi Ross, which recipe is failed, please ?

// Robert

> The problem is that people are using gnome.bbclass which is a kitchen
> sink class, inheriting many other classes that may or may not be used.
>   To resolve this warning gnome.bbclass should extend the sanity check
> whitelist with --disable-schemas-install as it is the class causing
> gconf.bbclass to be inherited on recipes that don't use gconf.
>
> Ross
>
>

Re: [PATCH 1/2] gconf.bbclass: remove --disable-schemas-install

[Burton, Ross]

On 7 January 2014 13:05, Robert Yang <liezhi.yang@windriver.com> wrote:
>> *Some* packages have migrated away from gconf to gsettings.  Not all
>> packages have done this, so anything still using gconf is now broken.
>> Note that almost nothing in oe-core uses gsettings as we're still
>> using the GTK+ 2 stack generally.
>
> Hi Ross, which recipe is failed, please ?

A better question would be what package did you get this warning from?

As I said, any recipe that did inherit gnome and still used GConf
would be affected by your change.

Ross

Re: [PATCH 1/2] gconf.bbclass: remove --disable-schemas-install

[Koen Kooi]

Op 8 jan. 2014, om 17:12 heeft Burton, Ross <ross.burton@intel.com> het volgende geschreven:

> On 7 January 2014 13:05, Robert Yang <liezhi.yang@windriver.com> wrote:
>>> *Some* packages have migrated away from gconf to gsettings.  Not all
>>> packages have done this, so anything still using gconf is now broken.
>>> Note that almost nothing in oe-core uses gsettings as we're still
>>> using the GTK+ 2 stack generally.
>> 
>> Hi Ross, which recipe is failed, please ?
> 
> A better question would be what package did you get this warning from?
> 
> As I said, any recipe that did inherit gnome and still used GConf
> would be affected by your change.

Which is over half of the meta-gnome recipes :(

Re: [PATCH 1/2] gconf.bbclass: remove --disable-schemas-install

[Robert Yang]

On 01/09/2014 02:33 PM, Koen Kooi wrote:
>
> Op 8 jan. 2014, om 17:12 heeft Burton, Ross <ross.burton@intel.com> het volgende geschreven:
>
>> On 7 January 2014 13:05, Robert Yang <liezhi.yang@windriver.com> wrote:
>>>> *Some* packages have migrated away from gconf to gsettings.  Not all
>>>> packages have done this, so anything still using gconf is now broken.
>>>> Note that almost nothing in oe-core uses gsettings as we're still
>>>> using the GTK+ 2 stack generally.
>>>
>>> Hi Ross, which recipe is failed, please ?
>>
>> A better question would be what package did you get this warning from?
>>
>> As I said, any recipe that did inherit gnome and still used GConf
>> would be affected by your change.
>
> Which is over half of the meta-gnome recipes :(
>

Sounds bad, fortunately, it had been reverted.

// Robert

[PATCH 2/2] mime.bbclass: remove the --disable-update-mimedb

[Robert Yang]

Gnome and other freedesktop's mime data has been moved to
shared-mime-info, and there is no such an option any more, we already
have this recipe, so remove it, this will fix the warning:

configure was passed unrecognised options: --disable-update-mimedb

Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
---
 meta/classes/mime.bbclass | 2 --
 1 file changed, 2 deletions(-)

diff --git a/meta/classes/mime.bbclass b/meta/classes/mime.bbclass
index 690610e..721c73f 100644
--- a/meta/classes/mime.bbclass
+++ b/meta/classes/mime.bbclass
@@ -1,7 +1,5 @@
 DEPENDS += "shared-mime-info-native shared-mime-info"
 
-EXTRA_OECONF += "--disable-update-mimedb"
-
 mime_postinst() {
 if [ "$1" = configure ]; then
 	UPDATEMIMEDB=`which update-mime-database`
-- 
1.8.3.1