Re: [PATCH] Support installing systemd unit file.

2011-08-31 Thread Gaetan Nadon
On Wed, 2011-08-31 at 21:36 +0200, Michał Górny wrote: > On Wed, 31 Aug 2011 15:28:27 -0400 > Gaetan Nadon wrote: > > > On Wed, 2011-08-31 at 15:16 -0400, Gaetan Nadon wrote: > > > > > On Wed, 2011-08-31 at 18:02 +0200, Michał Górny wrote: > > > > > > > Fixes: https://bugs.freedesktop.org/sho

Re: [PATCH] Support installing systemd unit file.

2011-08-31 Thread Michał Górny
On Wed, 31 Aug 2011 15:28:27 -0400 Gaetan Nadon wrote: > On Wed, 2011-08-31 at 15:16 -0400, Gaetan Nadon wrote: > > > On Wed, 2011-08-31 at 18:02 +0200, Michał Górny wrote: > > > > > Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=38339 > > > Signed-off-by: Michał Górny > > > Reviewed-by:

Re: [PATCH] Support installing systemd unit file.

2011-08-31 Thread Gaetan Nadon
On Wed, 2011-08-31 at 15:16 -0400, Gaetan Nadon wrote: > On Wed, 2011-08-31 at 18:02 +0200, Michał Górny wrote: > > > Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=38339 > > Signed-off-by: Michał Górny > > Reviewed-by: Gaetan Nadon > > Tested-by: Gaetan Nadon > > > > Sorry, I had no

Re: [PATCH] Support installing systemd unit file.

2011-08-31 Thread Gaetan Nadon
On Wed, 2011-08-31 at 18:02 +0200, Michał Górny wrote: > Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=38339 > Signed-off-by: Michał Górny > Reviewed-by: Gaetan Nadon > Tested-by: Gaetan Nadon > --- > .gitignore |1 + > Makefile.am| 13 - > configure.ac | 21

[PATCH] Support installing systemd unit file.

2011-08-31 Thread Michał Górny
Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=38339 Signed-off-by: Michał Górny Reviewed-by: Gaetan Nadon Tested-by: Gaetan Nadon --- .gitignore |1 + Makefile.am| 13 - configure.ac | 21 + xdm.service.in |9 + 4 files changed

Re: [PATCH] Support installing systemd unit file.

2011-08-31 Thread Gaetan Nadon
On Wed, 2011-08-31 at 16:52 +0200, Michał Górny wrote: > Signed-off-by: Michał Górny > Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=38339 > --- > .gitignore |1 + > Makefile.am| 13 - > configure.ac | 21 + > xdm.service.in |9

[PATCH] Support installing systemd unit file.

2011-08-31 Thread Michał Górny
Signed-off-by: Michał Górny Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=38339 --- .gitignore |1 + Makefile.am| 13 - configure.ac | 21 + xdm.service.in |9 + 4 files changed, 43 insertions(+), 1 deletions(-) create mode 1006

Re: [PATCH] Support installing systemd unit file.

2011-08-29 Thread Gaetan Nadon
On Sun, 2011-08-28 at 09:49 +0200, Michał Górny wrote: > +if HAVE_SYSTEMD > +systemdsystemunit_DATA = xdm.service > + > +xdm.service: xdm.service.in > + $(AM_V_GEN)$(SED) -e 's|BINDIR|$(bindir)|g' < $< > $@ > + Almost forgot: add xdm.service at the bottom of the .gitignore file. If you do

Re: [PATCH] Support installing systemd unit file.

2011-08-29 Thread Michał Górny
On Sun, 28 Aug 2011 17:12:41 -0400 Gaetan Nadon wrote: > > +if HAVE_SYSTEMD > > +systemdsystemunit_DATA = xdm.service > > + > > +xdm.service: xdm.service.in > > + $(AM_V_GEN)$(SED) -e 's|BINDIR|$(bindir)|g' < $< > $@ > > + > > +MOSTLYCLEANFILES = xdm.service > > +endif HAVE_SYSTEMD > > +EXTRA_D

Re: [PATCH] Support installing systemd unit file.

2011-08-28 Thread Michał Górny
On Sun, 28 Aug 2011 17:53:07 +0600 "Alexander E. Patrakov" wrote: > 28.08.2011 13:49, Michał Górny wrote: > > > +[Unit] > > +Description=X-Window Display Manager on %I > > +Requires=dev-%i.device > > +After=dev-%i.device systemd-user-sessions.service > > + > > +[Service] > > +ExecStart=BINDIR/xd

Re: [PATCH] Support installing systemd unit file.

2011-08-28 Thread Gaetan Nadon
On Sun, 2011-08-28 at 17:12 -0400, Gaetan Nadon wrote: > On Sun, 2011-08-28 at 09:49 +0200, Michał Górny wrote: > > > Signed-off-by: Michał Górny > > Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=38339 > > --- > > Makefile.am| 10 ++ > > configure.ac | 12 >

Re: [PATCH] Support installing systemd unit file.

2011-08-28 Thread Gaetan Nadon
On Sun, 2011-08-28 at 23:43 +0200, Julien Cristau wrote: > On Sun, Aug 28, 2011 at 17:12:41 -0400, Gaetan Nadon wrote: > > > In essence, when the user input clearly indicates he wants to set > > systemdsystemunitdir, an error should occur if systemd is not installed. > > > Err, I don't think tha

Re: [PATCH] Support installing systemd unit file.

2011-08-28 Thread Julien Cristau
On Sun, Aug 28, 2011 at 17:12:41 -0400, Gaetan Nadon wrote: > In essence, when the user input clearly indicates he wants to set > systemdsystemunitdir, an error should occur if systemd is not installed. > Err, I don't think that makes sense. I should be able to build xdm without systemd installe

Re: [PATCH] Support installing systemd unit file.

2011-08-28 Thread Gaetan Nadon
On Sun, 2011-08-28 at 09:49 +0200, Michał Górny wrote: > Signed-off-by: Michał Górny > Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=38339 > --- > Makefile.am| 10 ++ > configure.ac | 12 > xdm.service.in | 10 ++ > 3 files changed, 32 insertions(+)

Re: [PATCH] Support installing systemd unit file.

2011-08-28 Thread Michał Górny
On Sun, 28 Aug 2011 12:15:14 -0400 Gaetan Nadon wrote: > > Signed-off-by: Michał Górny > > Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=38339 > > --- > > Makefile.am| 10 ++ > > configure.ac | 12 > > xdm.service.in | 10 ++ > > 3 files changed, 3

Re: [PATCH] Support installing systemd unit file.

2011-08-28 Thread Gaetan Nadon
On Sun, 2011-08-28 at 09:49 +0200, Michał Górny wrote: > Signed-off-by: Michał Górny > Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=38339 > --- > Makefile.am| 10 ++ > configure.ac | 12 > xdm.service.in | 10 ++ > 3 files changed, 32 insertions(+)

Re: [PATCH] Support installing systemd unit file.

2011-08-28 Thread Julien Cristau
On Sun, Aug 28, 2011 at 17:53:07 +0600, Alexander E. Patrakov wrote: > 28.08.2011 13:49, Michał Górny wrote: > > >+[Unit] > >+Description=X-Window Display Manager on %I > >+Requires=dev-%i.device > >+After=dev-%i.device systemd-user-sessions.service > >+ > >+[Service] > >+ExecStart=BINDIR/xdm -no

Re: [PATCH] Support installing systemd unit file.

2011-08-28 Thread Alexander E. Patrakov
28.08.2011 13:49, Michał Górny wrote: +[Unit] +Description=X-Window Display Manager on %I +Requires=dev-%i.device +After=dev-%i.device systemd-user-sessions.service + +[Service] +ExecStart=BINDIR/xdm -nodaemon This deserves at least a comment why the X server is started on tty7 (btw, does it?

[PATCH] Support installing systemd unit file.

2011-08-28 Thread Michał Górny
Signed-off-by: Michał Górny Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=38339 --- Makefile.am| 10 ++ configure.ac | 12 xdm.service.in | 10 ++ 3 files changed, 32 insertions(+), 0 deletions(-) create mode 100644 xdm.service.in diff --git a/Make