[PATCH 1/1] xserver-xorg: Update to 1.16.3

[PATCH 1/1] xserver-xorg: Update to 1.16.3

[Aníbal Limón]

Add packageconfig for systemd-logind instead of hard disable in
EXTRA_OECONF this allows users to use packageconfig.

Update checksums.

Signed-off-by: Aníbal Limón <anibal.limon@linux.intel.com>
---
 meta/recipes-graphics/xorg-xserver/xserver-xorg.inc                   | 2 +-
 .../xorg-xserver/{xserver-xorg_1.16.2.bb => xserver-xorg_1.16.3.bb}   | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)
 rename meta/recipes-graphics/xorg-xserver/{xserver-xorg_1.16.2.bb => xserver-xorg_1.16.3.bb} (88%)

diff --git a/meta/recipes-graphics/xorg-xserver/xserver-xorg.inc b/meta/recipes-graphics/xorg-xserver/xserver-xorg.inc
index 6154295..ece7ddf 100644
--- a/meta/recipes-graphics/xorg-xserver/xserver-xorg.inc
+++ b/meta/recipes-graphics/xorg-xserver/xserver-xorg.inc
@@ -114,7 +114,6 @@ EXTRA_OECONF += "--with-fop=no \
                  --with-xkb-output=/var/lib/xkb \
                  --with-sha1=libcrypto \
                  ac_cv_file__usr_share_sgml_X11_defs_ent=no \
-                 --disable-systemd-logind \
 "
 
 PACKAGECONFIG ??= "udev ${@bb.utils.contains('DISTRO_FEATURES', 'opengl', 'dri dri2 glx', '', d)}"
@@ -128,6 +127,7 @@ PACKAGECONFIG[glx] = "--enable-glx --enable-glx-tls,--disable-glx,glproto virtua
 PACKAGECONFIG[unwind] = "--enable-libunwind,--disable-libunwind,libunwind"
 PACKAGECONFIG[xshmfence] = "--enable-xshmfence,--disable-xshmfence,libxshmfence"
 PACKAGECONFIG[xmlto] = "--with-xmlto, --without-xmlto, xmlto-native docbook-xml-dtd4-native docbook-xsl-stylesheets-native"
+PACKAGECONFIG[systemd-logind] = "--enable-systemd-logind=yes,--enable-systemd-logind=no,dbus,dbus-lib"
 
 do_install_append () {
 	# Its assumed base-files creates this for us
diff --git a/meta/recipes-graphics/xorg-xserver/xserver-xorg_1.16.2.bb b/meta/recipes-graphics/xorg-xserver/xserver-xorg_1.16.3.bb
similarity index 88%
rename from meta/recipes-graphics/xorg-xserver/xserver-xorg_1.16.2.bb
rename to meta/recipes-graphics/xorg-xserver/xserver-xorg_1.16.3.bb
index 7d7d83e..347ce3b 100644
--- a/meta/recipes-graphics/xorg-xserver/xserver-xorg_1.16.2.bb
+++ b/meta/recipes-graphics/xorg-xserver/xserver-xorg_1.16.3.bb
@@ -7,8 +7,8 @@ SRC_URI += "file://fix_open_max_preprocessor_error.patch \
             file://Fix-subwindow-in-Xi-emulated-events.patch \
            "
 
-SRC_URI[md5sum] = "89620960b13515db8d0a8dbb92a1378a"
-SRC_URI[sha256sum] = "446e0c3ebd556aced78ec0000ba9ae73f1e5317117d497f827afba48b787ce64"
+SRC_URI[md5sum] = "afd93977235584a9caa7528a737c1b52"
+SRC_URI[sha256sum] = "5e0f443238af1078b48f6eea98a382861b59187da221c2cf714d31c1d560b0fb"
 
 # These extensions are now integrated into the server, so declare the migration
 # path for in-place upgrades.
-- 
1.9.1

Re: [PATCH 1/1] xserver-xorg: Update to 1.16.3

[Burton, Ross]

[-- Attachment #1: Type: text/plain, Size: 392 bytes --]

Hi Anibal,

On 10 February 2015 at 15:45, Aníbal Limón <anibal.limon@linux.intel.com>
wrote:

> +PACKAGECONFIG[systemd-logind] =
> "--enable-systemd-logind=yes,--enable-systemd-logind=no,dbus,dbus-lib"
>

This adds an explicit runtime dependency on dbus-lib if logind is enabled,
which probably isn't what you want as the xserver will add this dependency
automatically.

Ross

[-- Attachment #2: Type: text/html, Size: 838 bytes --]

Re: [PATCH 1/1] xserver-xorg: Update to 1.16.3

[Aníbal Limón]

Hi Ross,

When you enable systemd-logind xserver automatically links to dbus 
causing a QA issue that is the reason of
add this dependency.

systemd uses dbus for all the communications with applications.

Cheers,
     alimon

On 10/02/15 09:52, Burton, Ross wrote:
> Hi Anibal,
>
> On 10 February 2015 at 15:45, Aníbal Limón <anibal.limon@linux.intel.com>
> wrote:
>
>> +PACKAGECONFIG[systemd-logind] =
>> "--enable-systemd-logind=yes,--enable-systemd-logind=no,dbus,dbus-lib"
>>
> This adds an explicit runtime dependency on dbus-lib if logind is enabled,
> which probably isn't what you want as the xserver will add this dependency
> automatically.
>
> Ross
>

Re: [PATCH 1/1] xserver-xorg: Update to 1.16.3

[Burton, Ross]

[-- Attachment #1: Type: text/plain, Size: 445 bytes --]

On 10 February 2015 at 15:57, Aníbal Limón <anibal.limon@linux.intel.com>
wrote:

> When you enable systemd-logind xserver automatically links to dbus causing
> a QA issue that is the reason of
> add this dependency.
>
> systemd uses dbus for all the communications with applications.
>

That explains the third option (additional build dependency of "dbus") but
not the fourth (additional runtime dependency of dbus-lib).

Ross

[-- Attachment #2: Type: text/html, Size: 868 bytes --]

Re: [PATCH 1/1] xserver-xorg: Update to 1.16.3

[Aníbal Limón]

On 10/02/15 09:58, Burton, Ross wrote:
> On 10 February 2015 at 15:57, Aníbal Limón <anibal.limon@linux.intel.com>
> wrote:
>
>> When you enable systemd-logind xserver automatically links to dbus causing
>> a QA issue that is the reason of
>> add this dependency.
>>
>> systemd uses dbus for all the communications with applications.
>>
> That explains the third option (additional build dependency of "dbus") but
> not the fourth (additional runtime dependency of dbus-lib).\
That's right, sending v2.

     alimon
>
> Ross
>