Remove all traces of HAL from gdm and remove the HAL dep, fixes bug 295206
authorNirbheek Chauhan <nirbheek@gentoo.org>
Sun, 7 Mar 2010 14:40:02 +0000 (14:40 +0000)
committerNirbheek Chauhan <nirbheek@gentoo.org>
Sun, 7 Mar 2010 14:40:02 +0000 (14:40 +0000)
Package-Manager: portage-2.1.8.1/cvs/Linux i686

gnome-base/gdm/ChangeLog
gnome-base/gdm/Manifest
gnome-base/gdm/files/gdm-2.28.2-remove-hal.patch [new file with mode: 0644]
gnome-base/gdm/gdm-2.28.2-r1.ebuild [new file with mode: 0644]

index e49a45c9144424310fbcad71b81eb35ee1f2c058..6890fd205f377f9cab0eaef88728582dd4bc3bc6 100644 (file)
@@ -1,6 +1,12 @@
 # ChangeLog for gnome-base/gdm
 # Copyright 1999-2010 Gentoo Foundation; Distributed under the GPL v2
-# $Header: /var/cvsroot/gentoo-x86/gnome-base/gdm/ChangeLog,v 1.316 2010/03/05 10:27:08 nirbheek Exp $
+# $Header: /var/cvsroot/gentoo-x86/gnome-base/gdm/ChangeLog,v 1.317 2010/03/07 14:40:01 nirbheek Exp $
+
+*gdm-2.28.2-r1 (07 Mar 2010)
+
+  07 Mar 2010; Nirbheek Chauhan <nirbheek@gentoo.org> +gdm-2.28.2-r1.ebuild,
+  +files/gdm-2.28.2-remove-hal.patch:
+  Remove all traces of HAL from gdm and remove the HAL dep, fixes bug 295206
 
   05 Mar 2010; Nirbheek Chauhan <nirbheek@gentoo.org> gdm-2.28.2.ebuild:
   Add dep on xdg-utils, fixes bug 302135
index 02c88eef3b55562b3d89fc7d77ff7dd26792a755..12a2c492e3a2b295298f55af551d2077aff8d729 100644 (file)
@@ -8,6 +8,7 @@ AUX gdm-2.20.10-xinitrc-ssh-agent.patch 704 RMD160 cc84a57a53e0f8ab8688173f96869
 AUX gdm-2.20.2-gksu.patch 544 RMD160 044068ec9c0fac089dcfa8b62597741d6bf6a4ac SHA1 7672aa94ea512cf50740466c6bc072fa236c7d01 SHA256 3c7dabba9904d1ae382989099e03938c47eb2ce882f50d8fae9164a573bddb4d
 AUX gdm-2.20.9-parallel-make.patch 1499 RMD160 aaca98cd72b464aec043b2aeb51973f9ab3c9a33 SHA1 1718004f16455a3f3c957b40b2391c563988681e SHA256 c8d04d1cae1c2bac16bf3f41fd481eacf5f4a3962f661996f93da39d4161bb0d
 AUX gdm-2.26.1-libxklavier4.patch 1842 RMD160 cae9308944ff4118bd06e411b11e1df6b97fdf99 SHA1 0fbb97f83b980c9bfaab1e95108d9462640c9134 SHA256 7978417318e80628939cdbf62d59ba56cb15242748cb89b5a8b6c02b5265173a
+AUX gdm-2.28.2-remove-hal.patch 6934 RMD160 94083bac8b9b489d47414330cd06601c32a3901c SHA1 0a254daba5dd8c9ee91d362ee64d185c521bf8fa SHA256 a8303f2834132c88c6cd6925dd96a10877c7a180c761246ac701f1dd09de5d7d
 DIST gdm-2.20.10.tar.bz2 3940077 RMD160 4bb50375d30638143b47b7884405e9999e304da5 SHA1 42e9e52184393352f005f1735843a30f764aef18 SHA256 5d4a2e5973a778871c9bfb91d76580747a4ef40771fd4368169a5868bb2db127
 DIST gdm-2.20.9-gentoo-files-r1.tar.bz2 3494 RMD160 9d8f9954bd0d8b657a2faf2516b4548b9f1556d3 SHA1 28ceedb42440fdb50dffa7e69b18f213b00e3935 SHA256 666c0bf1ba28ccc257bf1235b5286e1eb9e15230c0036413ab428e09c54238f9
 DIST gdm-2.26-gentoo-patches.tar.bz2 4175 RMD160 356cd339b733e41c504bf2c951a322da205439be SHA1 a58adde0bb70b56d4b61cd5bf0ca81fbbfa44c69 SHA256 e6ad1a4995eb622cf74e5e897ac0588dff122446af5e1e52f165fd8bccb105d2
@@ -17,6 +18,7 @@ DIST gentoo-gdm-theme-r3.tar.bz2 411703 RMD160 9188c8f2661163be72595a6b4924fae06
 EBUILD gdm-2.20.10-r2.ebuild 6714 RMD160 51ec36566d48b4a565faf9c91770b084a87901be SHA1 058e00a26aab1642d1f54c95b9e7ce3dcf1aab39 SHA256 96af7b13bd13537c337cf6c82e85488a821940803984ab0d497753c0dd9a8fb2
 EBUILD gdm-2.20.10-r3.ebuild 6870 RMD160 0ba86bd89d1705d2636a7ad54aae5e659751d5ce SHA1 26fba0a993d340bbc978748138eb9631f7eaac68 SHA256 b482a4ad944563d4e32fec54f769e16e17cf428708565dff17bbdd249c7a4f33
 EBUILD gdm-2.26.1.ebuild 5658 RMD160 af5494ccebe8a176f12d59f201eda2df97bbe152 SHA1 34d31f15cb2ebcf374234918554db028424dd715 SHA256 b226d4231faee7063773054bb53c45d44ce0fdad1e3da893bae7e70eb6d41ca4
+EBUILD gdm-2.28.2-r1.ebuild 5832 RMD160 83b55fd152c606bea1a4de5351099b38c9f5d442 SHA1 9ddc0776c4597fcc3b41934fd20349581c964ef5 SHA256 b719cb724e6deb200ba64daf4d0e7ad8f43f10f3aa7d20c0e00b4477f18af57e
 EBUILD gdm-2.28.2.ebuild 5762 RMD160 4d63704daf128ea3573f126223a6fdd12f32f2e9 SHA1 a09b76bc59f5e5e2f7c40c33ddb98ffdf7e4c368 SHA256 5400ebee80d05044c697450d19fa6edaef24022e1f94645532b36eb4678e455e
-MISC ChangeLog 43269 RMD160 f5bdd53440bcfea09ea0495e4a3203998c5ef4e1 SHA1 a60ef729b12b493f676ad7838fbd2574a1d19311 SHA256 d18728fed486af3e46052640f515321d407822192c91b383d9fb4622b42df285
+MISC ChangeLog 43492 RMD160 0ef9c1a9cbff94e7378fd3856fabb1b8fe91eab1 SHA1 0ec07ebd8e0e8293b6b8d7147475a50926bad0fa SHA256 34f724ec5c0c009833ba50c27a92cfb16db4f920780924000aa55fd2e07f43ea
 MISC metadata.xml 536 RMD160 53b21d3083b79fe916434c4031a06d108130185c SHA1 b1e22aa4ac145b16471c105709861ffff6585795 SHA256 9d8a22879d6e19a12a8f47ed551cdf440fec0d7057ff0a12feec1c5752c9ce2d
diff --git a/gnome-base/gdm/files/gdm-2.28.2-remove-hal.patch b/gnome-base/gdm/files/gdm-2.28.2-remove-hal.patch
new file mode 100644 (file)
index 0000000..e72a30f
--- /dev/null
@@ -0,0 +1,194 @@
+Remove all traces of HAL. The last hunk which removes test-hal-seats.c has been
+deleted to save space; the file is manually removed instead.
+
+http://bugs.gentoo.org/show_bug.cgi?id=295206
+---
+From 3daaac264df2f5a587c35cd8e41ecf18900fa9c9 Mon Sep 17 00:00:00 2001
+From: Martin Pitt <martin.pitt@ubuntu.com>
+Date: Thu, 21 Jan 2010 00:00:02 +0100
+Subject: [PATCH] remove remaining traces of Hal
+
+The code which used the Hal connection already was disabled, so gdm connected
+to Hal in vain (which just triggered Hal startup when using D-Bus activation).
+Remove all remaining traces of hal now.
+
+https://bugzilla.gnome.org/show_bug.cgi?id=593787
+---
+ daemon/Makefile.am                 |    9 --
+ daemon/gdm-local-display-factory.c |  110 -----------------------
+ daemon/test-hal-seats.c            |  173 ------------------------------------
+ 3 files changed, 0 insertions(+), 293 deletions(-)
+ delete mode 100644 daemon/test-hal-seats.c
+
+diff --git a/daemon/Makefile.am b/daemon/Makefile.am
+index a122a15..79bb63e 100644
+--- a/daemon/Makefile.am
++++ b/daemon/Makefile.am
+@@ -75,18 +75,9 @@ gdm-product-display-glue.h: gdm-product-display.xml Makefile.am
+       dbus-binding-tool --prefix=gdm_product_display --mode=glib-server --output=gdm-product-display-glue.h $(srcdir)/gdm-product-display.xml
+ 
+ noinst_PROGRAMS =             \
+-      test-hal-seats          \
+       test-session            \
+       $(NULL)
+ 
+-test_hal_seats_SOURCES =      \
+-      test-hal-seats.c        \
+-      $(NULL)
+-
+-test_hal_seats_LDADD =                \
+-      $(DAEMON_LIBS)          \
+-      $(NULL)
+-
+ test_session_SOURCES =                \
+       test-session.c          \
+       gdm-session.c           \
+diff --git a/daemon/gdm-local-display-factory.c b/daemon/gdm-local-display-factory.c
+index 3984ada..69daf9e 100644
+--- a/daemon/gdm-local-display-factory.c
++++ b/daemon/gdm-local-display-factory.c
+@@ -45,12 +45,6 @@
+ #define GDM_LOCAL_DISPLAY_FACTORY_DBUS_PATH GDM_DBUS_PATH "/LocalDisplayFactory"
+ #define GDM_MANAGER_DBUS_NAME               "org.gnome.DisplayManager.LocalDisplayFactory"
+ 
+-#define HAL_DBUS_NAME                           "org.freedesktop.Hal"
+-#define HAL_DBUS_MANAGER_PATH                   "/org/freedesktop/Hal/Manager"
+-#define HAL_DBUS_MANAGER_INTERFACE              "org.freedesktop.Hal.Manager"
+-#define HAL_DBUS_DEVICE_INTERFACE               "org.freedesktop.Hal.Device"
+-#define SEAT_PCI_DEVICE_CLASS                   3
+-
+ #define MAX_DISPLAY_FAILURES 5
+ 
+ struct GdmLocalDisplayFactoryPrivate
+@@ -372,83 +366,6 @@ create_display (GdmLocalDisplayFactory *factory)
+         return display;
+ }
+ 
+-#if 0
+-static void
+-create_display_for_device (GdmLocalDisplayFactory *factory,
+-                           DBusGProxy             *device_proxy)
+-{
+-        create_display (factory);
+-}
+-
+-static void
+-create_displays_for_pci_devices (GdmLocalDisplayFactory *factory)
+-{
+-        char      **devices;
+-        const char *key;
+-        const char *value;
+-        GError     *error;
+-        gboolean    res;
+-        int         i;
+-
+-        g_debug ("GdmLocalDisplayFactory: Getting PCI seat devices");
+-
+-        key = "info.bus";
+-        value = "pci";
+-
+-        devices = NULL;
+-        error = NULL;
+-        res = dbus_g_proxy_call (factory->priv->proxy,
+-                                 "FindDeviceStringMatch",
+-                                 &error,
+-                                 G_TYPE_STRING, key,
+-                                 G_TYPE_STRING, value,
+-                                 G_TYPE_INVALID,
+-                                 G_TYPE_STRV, &devices,
+-                                 G_TYPE_INVALID);
+-        if (! res) {
+-                g_warning ("Unable to query HAL: %s", error->message);
+-                g_error_free (error);
+-        }
+-
+-        /* now look for pci class 3 */
+-        key = "pci.device_class";
+-        for (i = 0; devices [i] != NULL; i++) {
+-                DBusGProxy *device_proxy;
+-                int         class_val;
+-
+-                device_proxy = dbus_g_proxy_new_for_name (factory->priv->connection,
+-                                                          HAL_DBUS_NAME,
+-                                                          devices [i],
+-                                                          HAL_DBUS_DEVICE_INTERFACE);
+-                if (device_proxy == NULL) {
+-                        continue;
+-                }
+-
+-                error = NULL;
+-                res = dbus_g_proxy_call (device_proxy,
+-                                         "GetPropertyInteger",
+-                                         &error,
+-                                         G_TYPE_STRING, key,
+-                                         G_TYPE_INVALID,
+-                                         G_TYPE_INT, &class_val,
+-                                         G_TYPE_INVALID);
+-                if (! res) {
+-                        g_warning ("Unable to query HAL: %s", error->message);
+-                        g_error_free (error);
+-                }
+-
+-                if (class_val == SEAT_PCI_DEVICE_CLASS) {
+-                        g_debug ("GdmLocalDisplayFactory: Found device: %s", devices [i]);
+-                        create_display_for_device (factory, device_proxy);
+-                }
+-
+-                g_object_unref (device_proxy);
+-        }
+-
+-        g_strfreev (devices);
+-}
+-#endif
+-
+ static gboolean
+ gdm_local_display_factory_start (GdmDisplayFactory *base_factory)
+ {
+@@ -525,29 +442,6 @@ register_factory (GdmLocalDisplayFactory *factory)
+         return TRUE;
+ }
+ 
+-static gboolean
+-connect_to_hal (GdmLocalDisplayFactory *factory)
+-{
+-        factory->priv->proxy = dbus_g_proxy_new_for_name (factory->priv->connection,
+-                                                          HAL_DBUS_NAME,
+-                                                          HAL_DBUS_MANAGER_PATH,
+-                                                          HAL_DBUS_MANAGER_INTERFACE);
+-        if (factory->priv->proxy == NULL) {
+-                g_warning ("Couldn't create proxy for HAL Manager");
+-                return FALSE;
+-        }
+-
+-        return TRUE;
+-}
+-
+-static void
+-disconnect_from_hal (GdmLocalDisplayFactory *factory)
+-{
+-        if (factory->priv->proxy == NULL) {
+-                g_object_unref (factory->priv->proxy);
+-        }
+-}
+-
+ static GObject *
+ gdm_local_display_factory_constructor (GType                  type,
+                                        guint                  n_construct_properties,
+@@ -565,8 +459,6 @@ gdm_local_display_factory_constructor (GType                  type,
+                 g_warning ("Unable to register local display factory with system bus");
+         }
+ 
+-        connect_to_hal (factory);
+-
+         return G_OBJECT (factory);
+ }
+ 
+@@ -611,8 +503,6 @@ gdm_local_display_factory_finalize (GObject *object)
+ 
+         g_hash_table_destroy (factory->priv->displays);
+ 
+-        disconnect_from_hal (factory);
+-
+         G_OBJECT_CLASS (gdm_local_display_factory_parent_class)->finalize (object);
+ }
+ 
diff --git a/gnome-base/gdm/gdm-2.28.2-r1.ebuild b/gnome-base/gdm/gdm-2.28.2-r1.ebuild
new file mode 100644 (file)
index 0000000..fef18fa
--- /dev/null
@@ -0,0 +1,205 @@
+# Copyright 1999-2010 Gentoo Foundation
+# Distributed under the terms of the GNU General Public License v2
+# $Header: /var/cvsroot/gentoo-x86/gnome-base/gdm/gdm-2.28.2-r1.ebuild,v 1.1 2010/03/07 14:40:01 nirbheek Exp $
+
+EAPI="2"
+
+inherit eutils pam gnome2 autotools
+
+DESCRIPTION="GNOME Display Manager"
+HOMEPAGE="http://www.gnome.org/projects/gdm/"
+
+LICENSE="GPL-2"
+SLOT="0"
+KEYWORDS="~amd64 ~x86"
+
+IUSE_LIBC="elibc_glibc"
+IUSE="accessibility +consolekit debug ipv6 gnome-keyring selinux tcpd test xinerama +xklavier $IUSE_LIBC"
+
+# Name of the tarball with gentoo specific files
+GDM_EXTRA="${PN}-2.20.9-gentoo-files-r1"
+
+SRC_URI="${SRC_URI}
+       mirror://gentoo/${PN}-2.26-gentoo-patches.tar.bz2
+       mirror://gentoo/${GDM_EXTRA}.tar.bz2"
+
+# FIXME: gdm has a strange behaviour on reboot (from runlevel), especially when xdm tries to stop it (its blocks).
+# NOTE: x11-base/xorg-server dep is for X_SERVER_PATH etc, bug 295686
+RDEPEND=">=sys-apps/devicekit-power-008
+       >=dev-libs/dbus-glib-0.74
+       >=dev-libs/glib-2.15.4
+       >=x11-libs/gtk+-2.10.0
+       >=x11-libs/pango-1.3
+       >=media-libs/libcanberra-0.4[gtk]
+       >=gnome-base/libglade-2
+       >=gnome-base/gconf-2.6.1
+       >=gnome-base/gnome-panel-2
+       >=x11-misc/xdg-utils-1.0.2-r3
+
+       app-text/iso-codes
+
+       x11-base/xorg-server
+       x11-libs/libXi
+       x11-libs/libXau
+       x11-libs/libX11
+       x11-libs/libXdmcp
+       x11-libs/libXext
+       x11-libs/libXft
+       x11-apps/sessreg
+
+       virtual/pam
+       consolekit? ( sys-auth/consolekit )
+
+       accessibility? ( x11-libs/libXevie )
+       gnome-keyring? ( >=gnome-base/gnome-keyring-2.22[pam] )
+       selinux? ( sys-libs/libselinux )
+       tcpd? ( >=sys-apps/tcp-wrappers-7.6 )
+       xinerama? ( x11-libs/libXinerama )
+       xklavier? ( >=x11-libs/libxklavier-4 )
+
+       !gnome-extra/fast-user-switch-applet"
+DEPEND="${RDEPEND}
+       test? ( >=dev-libs/check-0.9.4 )
+       xinerama? ( x11-proto/xineramaproto )
+       sys-devel/gettext
+       x11-proto/inputproto
+       >=dev-util/intltool-0.40
+       >=dev-util/pkgconfig-0.19
+       >=app-text/scrollkeeper-0.1.4
+       >=app-text/gnome-doc-utils-0.3.2"
+PDEPEND=">=sys-auth/pambase-20090430[consolekit=,gnome-keyring=]"
+
+DOCS="AUTHORS ChangeLog NEWS README TODO"
+
+pkg_setup() {
+       # PAM is the only auth scheme supported
+       # even though configure lists shadow and crypt
+       # they don't have any corresponding code
+       G2CONF="${G2CONF}
+               --disable-schemas-install
+               --localstatedir=/var
+               --with-xdmcp=yes
+               --enable-authentication-scheme=pam
+               --with-pam-prefix=/etc
+               SOUND_PROGRAM=/usr/bin/gdmplay
+               $(use_with accessibility xevie)
+               $(use_enable debug)
+               $(use_enable ipv6)
+               $(use_enable xklavier libxklavier)
+               $(use_with consolekit console-kit)
+               $(use_with selinux)
+               $(use_with tcpd tcp-wrappers)
+               $(use_with xinerama)"
+
+       enewgroup gdm
+       enewuser gdm -1 -1 /var/lib/gdm gdm
+}
+
+src_prepare() {
+       gnome2_src_prepare
+
+       # remove unneeded linker directive for selinux (#41022)
+       epatch "${WORKDIR}/${PN}-2.26.1-selinux-remove-attr.patch"
+
+       # Make it daemonize so that the boot process can continue (#236701)
+       epatch "${WORKDIR}/${PN}-2.26.1-fix-daemonize-regression.patch"
+
+       # Fix VT grab problem causing GDM to grab VT2 instead of 7 (#261339)
+       epatch "${WORKDIR}/${PN}-2.26.1-broken-VT-detection.patch"
+
+       # Make custom session work, bug #.
+       epatch "${WORKDIR}/${PN}-2.26.1-custom-session.patch"
+
+       # ssh-agent handling must be done at xinitrc.d
+       epatch "${WORKDIR}/${PN}-2.26.1-xinitrc-ssh-agent.patch"
+
+       # Fix libxklavier automagic support
+       epatch "${WORKDIR}/${PN}-2.26.1-automagic-libxklavier-support.patch"
+
+       # Remove all traces of HAL
+       epatch "${FILESDIR}/${P}-remove-hal.patch"
+       rm -vf "${S}/daemon/test-hal-seats.c"
+
+       intltoolize --force --copy --automake || die "intltoolize failed"
+       eautoreconf
+}
+
+src_install() {
+       gnome2_src_install
+
+       local gentoodir="${WORKDIR}/${GDM_EXTRA}"
+
+       # gdm-binary should be gdm to work with our init (#5598)
+       rm -f "${D}/usr/sbin/gdm"
+       dosym /usr/sbin/gdm-binary /usr/sbin/gdm
+
+       # our x11's scripts point to /usr/bin/gdm
+       dosym /usr/sbin/gdm-binary /usr/bin/gdm
+
+       # log, etc.
+       keepdir /var/log/gdm
+       keepdir /var/gdm
+
+       fowners root:gdm /var/gdm
+       fperms 1770 /var/gdm
+
+       # add a custom xsession .desktop by default (#44537)
+       exeinto /etc/X11/dm/Sessions
+       doexe "${gentoodir}/custom.desktop" || die "doexe 1 failed"
+
+       # add xinitrc.d scripts
+       exeinto /etc/X11/xinit/xinitrc.d
+       doexe "${FILESDIR}/49-keychain" || die "doexe 2 failed"
+       doexe "${FILESDIR}/50-ssh-agent" || die "doexe 3 failed"
+
+       # install XDG_DATA_DIRS gdm changes
+       echo 'XDG_DATA_DIRS="/usr/share/gdm"' > 99xdg-gdm
+       doenvd 99xdg-gdm || die "doenvd failed"
+
+       # add a custom sound playing script (#248253)
+       dobin "${gentoodir}/gdmplay"
+
+       # avoid file collision, bug #213118
+       rm -f "${D}/usr/share/xsessions/gnome.desktop"
+
+       # We replace the pam stuff by our own
+       rm -rf "${D}/etc/pam.d"
+
+       use gnome-keyring && sed -i "s:#Keyring=::g" "${gentoodir}"/pam.d/*
+
+       dopamd "${gentoodir}"/pam.d/*
+       dopamsecurity console.apps "${gentoodir}/security/console.apps/gdmsetup"
+}
+
+pkg_postinst() {
+       gnome2_pkg_postinst
+
+       ewarn
+       ewarn "This is an EXPERIMENTAL release, please bear with its bugs and"
+       ewarn "visit us on #gentoo-desktop if you have problems."
+       ewarn
+
+       elog "To make GDM start at boot, edit /etc/conf.d/xdm"
+       elog "and then execute 'rc-update add xdm default'."
+       elog "If you already have GDM running, you will need to restart it."
+
+       if use gnome-keyring; then
+               elog "For autologin to unlock your keyring, you need to set an empty"
+               elog "password on your keyring. Use app-crypt/seahorse for that."
+       fi
+
+       if [ -f "/etc/X11/gdm/gdm.conf" ]; then
+               elog "You had /etc/X11/gdm/gdm.conf which is the old configuration"
+               elog "file.  It has been moved to /etc/X11/gdm/gdm-pre-gnome-2.16"
+               mv /etc/X11/gdm/gdm.conf /etc/X11/gdm/gdm-pre-gnome-2.16
+       fi
+}
+
+pkg_postrm() {
+       gnome2_pkg_postrm
+
+       if [[ "$(rc-config list default | grep xdm)" != "" ]] ; then
+               elog "To remove GDM from startup please execute"
+               elog "'rc-update del xdm default'"
+       fi
+}