Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH 1/1 v3] allow xserver without the systemd login daemon.
@ 2015-03-31 20:41 Sagaert Johan
  2015-04-07 20:28 ` Thomas Petazzoni
  0 siblings, 1 reply; 2+ messages in thread
From: Sagaert Johan @ 2015-03-31 20:41 UTC (permalink / raw)
  To: buildroot


Signed-off-by: Sagaert Johan <sagaert.johan@proximus.be>
---
Patch v2->v3 :
	added dependency as suggested by Arnout
	  (configure does pkg-config check for libsystemd-daemon)
---
Patch v1->v2 :
	added else clause as suggested by Thomas.
---
 package/x11r7/xserver_xorg-server/xserver_xorg-server.mk | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/package/x11r7/xserver_xorg-server/xserver_xorg-server.mk b/package/x11r7/xserver_xorg-server/xserver_xorg-server.mk
index bb5381c..2bab0f5 100644
--- a/package/x11r7/xserver_xorg-server/xserver_xorg-server.mk
+++ b/package/x11r7/xserver_xorg-server/xserver_xorg-server.mk
@@ -64,6 +64,17 @@ XSERVER_XORG_SERVER_CONF_OPTS = \
 	--with-fontrootdir=/usr/share/fonts/X11/ \
 	--$(if $(BR2_PACKAGE_XSERVER_XORG_SERVER_XVFB),en,dis)able-xvfb
 
+ifeq ($(BR2_PACKAGE_SYSTEMD),y)
+XSERVER_XORG_CONF_OPTS += \
+	--with-systemd-daemon \
+	--enable-systemd-logind
+XSERVER_XORG_SERVER_DEPENDENCIES += systemd
+else
+XSERVER_XORG_CONF_OPTS += \
+	--without-systemd-daemon \
+	--disable-systemd-logind
+endif
+
 ifeq ($(BR2_PACKAGE_XSERVER_XORG_SERVER_MODULAR),y)
 XSERVER_XORG_SERVER_CONF_OPTS += --enable-xorg
 XSERVER_XORG_SERVER_DEPENDENCIES += libpciaccess
-- 
2.3.3

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* [Buildroot] [PATCH 1/1 v3] allow xserver without the systemd login daemon.
  2015-03-31 20:41 [Buildroot] [PATCH 1/1 v3] allow xserver without the systemd login daemon Sagaert Johan
@ 2015-04-07 20:28 ` Thomas Petazzoni
  0 siblings, 0 replies; 2+ messages in thread
From: Thomas Petazzoni @ 2015-04-07 20:28 UTC (permalink / raw)
  To: buildroot

Dear Sagaert Johan,

On Tue, 31 Mar 2015 22:41:02 +0200, Sagaert Johan wrote:
> 
> Signed-off-by: Sagaert Johan <sagaert.johan@proximus.be>

Applied, after tweaking the commit log to include the name of the
affected package. Sagaert: all commit title should have the form:

	<package>: <description>

Note that systemd-logind support actually doesn't require systemd, it
only needs D-Bus and udev. But since it certainly doesn't make sense
without systemd in the system, I believe your patch is a good enough
solution.

Thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2015-04-07 20:28 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-03-31 20:41 [Buildroot] [PATCH 1/1 v3] allow xserver without the systemd login daemon Sagaert Johan
2015-04-07 20:28 ` Thomas Petazzoni

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox