Linux bluetooth development
 help / color / mirror / Atom feed
* [PATCH] Allow using obexd without systemd in the user session
@ 2013-10-12 15:45 Giovanni Campagna
  2013-10-12 16:08 ` Marcel Holtmann
  0 siblings, 1 reply; 6+ messages in thread
From: Giovanni Campagna @ 2013-10-12 15:45 UTC (permalink / raw)
  To: linux-bluetooth; +Cc: Giovanni Campagna

From: Giovanni Campagna <gcampagna@src.gnome.org>

Not all sessions run systemd --user (actually, the majority
doesn't), so the dbus daemon must be able to spawn obexd
directly, and to do so it needs the full path of the daemon.
---
 Makefile.obexd                      | 4 ++--
 obexd/src/org.bluez.obex.service    | 4 ----
 obexd/src/org.bluez.obex.service.in | 4 ++++
 3 files changed, 6 insertions(+), 6 deletions(-)
 delete mode 100644 obexd/src/org.bluez.obex.service
 create mode 100644 obexd/src/org.bluez.obex.service.in

diff --git a/Makefile.obexd b/Makefile.obexd
index d5377cb..751e0e5 100644
--- a/Makefile.obexd
+++ b/Makefile.obexd
@@ -2,12 +2,12 @@
 if SYSTEMD
 systemduserunitdir = @SYSTEMD_USERUNITDIR@
 systemduserunit_DATA = obexd/src/obex.service
+endif
 
 dbussessionbusdir = @DBUS_SESSIONBUSDIR@
 dbussessionbus_DATA = obexd/src/org.bluez.obex.service
-endif
 
-EXTRA_DIST += obexd/src/obex.service.in obexd/src/org.bluez.obex.service
+EXTRA_DIST += obexd/src/obex.service.in obexd/src/org.bluez.obex.service.in
 
 obex_plugindir = $(libdir)/obex/plugins
 
diff --git a/obexd/src/org.bluez.obex.service b/obexd/src/org.bluez.obex.service
deleted file mode 100644
index a538088..0000000
--- a/obexd/src/org.bluez.obex.service
+++ /dev/null
@@ -1,4 +0,0 @@
-[D-BUS Service]
-Name=org.bluez.obex
-Exec=/bin/false
-SystemdService=dbus-org.bluez.obex.service
diff --git a/obexd/src/org.bluez.obex.service.in b/obexd/src/org.bluez.obex.service.in
new file mode 100644
index 0000000..9c815f2
--- /dev/null
+++ b/obexd/src/org.bluez.obex.service.in
@@ -0,0 +1,4 @@
+[D-BUS Service]
+Name=org.bluez.obex
+Exec=@libexecdir@/obexd
+SystemdService=dbus-org.bluez.obex.service
-- 
1.8.3.1


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

* Re: [PATCH] Allow using obexd without systemd in the user session
  2013-10-12 15:45 [PATCH] Allow using obexd without systemd in the user session Giovanni Campagna
@ 2013-10-12 16:08 ` Marcel Holtmann
  2013-10-12 16:18   ` Giovanni Campagna
  2013-10-12 16:25   ` Bastien Nocera
  0 siblings, 2 replies; 6+ messages in thread
From: Marcel Holtmann @ 2013-10-12 16:08 UTC (permalink / raw)
  To: Giovanni Campagna; +Cc: linux-bluetooth, Giovanni Campagna

Hi Giovanni,

> Not all sessions run systemd --user (actually, the majority
> doesn't), so the dbus daemon must be able to spawn obexd
> directly, and to do so it needs the full path of the daemon.
> ---
> Makefile.obexd                      | 4 ++--
> obexd/src/org.bluez.obex.service    | 4 ----
> obexd/src/org.bluez.obex.service.in | 4 ++++
> 3 files changed, 6 insertions(+), 6 deletions(-)
> delete mode 100644 obexd/src/org.bluez.obex.service
> create mode 100644 obexd/src/org.bluez.obex.service.in
> 
> diff --git a/Makefile.obexd b/Makefile.obexd
> index d5377cb..751e0e5 100644
> --- a/Makefile.obexd
> +++ b/Makefile.obexd
> @@ -2,12 +2,12 @@
> if SYSTEMD
> systemduserunitdir = @SYSTEMD_USERUNITDIR@
> systemduserunit_DATA = obexd/src/obex.service
> +endif
> 
> dbussessionbusdir = @DBUS_SESSIONBUSDIR@
> dbussessionbus_DATA = obexd/src/org.bluez.obex.service
> -endif
> 
> -EXTRA_DIST += obexd/src/obex.service.in obexd/src/org.bluez.obex.service
> +EXTRA_DIST += obexd/src/obex.service.in obexd/src/org.bluez.obex.service.in
> 
> obex_plugindir = $(libdir)/obex/plugins
> 
> diff --git a/obexd/src/org.bluez.obex.service b/obexd/src/org.bluez.obex.service
> deleted file mode 100644
> index a538088..0000000
> --- a/obexd/src/org.bluez.obex.service
> +++ /dev/null
> @@ -1,4 +0,0 @@
> -[D-BUS Service]
> -Name=org.bluez.obex
> -Exec=/bin/false
> -SystemdService=dbus-org.bluez.obex.service
> diff --git a/obexd/src/org.bluez.obex.service.in b/obexd/src/org.bluez.obex.service.in
> new file mode 100644
> index 0000000..9c815f2
> --- /dev/null
> +++ b/obexd/src/org.bluez.obex.service.in
> @@ -0,0 +1,4 @@
> +[D-BUS Service]
> +Name=org.bluez.obex
> +Exec=@libexecdir@/obexd
> +SystemdService=dbus-org.bluez.obex.service

that is something your distro specific package should be fixing if that is needed. The focus is on integration with systemd and nothing else. You can use --disable-systemd and provide your own set of D-Bus service files.

Regards

Marcel


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

* Re: [PATCH] Allow using obexd without systemd in the user session
  2013-10-12 16:08 ` Marcel Holtmann
@ 2013-10-12 16:18   ` Giovanni Campagna
  2013-10-12 16:25   ` Bastien Nocera
  1 sibling, 0 replies; 6+ messages in thread
From: Giovanni Campagna @ 2013-10-12 16:18 UTC (permalink / raw)
  To: Marcel Holtmann; +Cc: linux-bluetooth

2013/10/12 Marcel Holtmann <marcel@holtmann.org>:
> Hi Giovanni,
>
> [...]
>
> that is something your distro specific package should be fixing if that is needed. The focus is on integration with systemd and nothing else. You can use --disable-systemd and provide your own set of D-Bus service files.

I don't think shipping with upstream packages that require distro
patches in common configurations is a good idea (it breaks jhbuild
setups, for example), but ok, I filed
https://bugzilla.redhat.com/show_bug.cgi?id=1018496

Thanks for the quick answer,

Giovanni

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

* Re: [PATCH] Allow using obexd without systemd in the user session
  2013-10-12 16:08 ` Marcel Holtmann
  2013-10-12 16:18   ` Giovanni Campagna
@ 2013-10-12 16:25   ` Bastien Nocera
  2013-10-14 15:02     ` Luiz Augusto von Dentz
  1 sibling, 1 reply; 6+ messages in thread
From: Bastien Nocera @ 2013-10-12 16:25 UTC (permalink / raw)
  To: Marcel Holtmann; +Cc: Giovanni Campagna, linux-bluetooth, Giovanni Campagna

On Sat, 2013-10-12 at 18:08 +0200, Marcel Holtmann wrote:
> Hi Giovanni,
<snip>
> that is something your distro specific package should be fixing if
> that is needed. The focus is on integration with systemd and nothing
> else. You can use --disable-systemd and provide your own set of D-Bus
> service files.

There aren't any distributions that ship systemd for user sessions
though.

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

* Re: [PATCH] Allow using obexd without systemd in the user session
  2013-10-12 16:25   ` Bastien Nocera
@ 2013-10-14 15:02     ` Luiz Augusto von Dentz
  2013-10-14 18:02       ` Bastien Nocera
  0 siblings, 1 reply; 6+ messages in thread
From: Luiz Augusto von Dentz @ 2013-10-14 15:02 UTC (permalink / raw)
  To: Bastien Nocera
  Cc: Marcel Holtmann, Giovanni Campagna,
	linux-bluetooth@vger.kernel.org, Giovanni Campagna

Hi Bastien,

On Sat, Oct 12, 2013 at 7:25 PM, Bastien Nocera <hadess@hadess.net> wrote:
> On Sat, 2013-10-12 at 18:08 +0200, Marcel Holtmann wrote:
>> Hi Giovanni,
> <snip>
>> that is something your distro specific package should be fixing if
>> that is needed. The focus is on integration with systemd and nothing
>> else. You can use --disable-systemd and provide your own set of D-Bus
>> service files.
>
> There aren't any distributions that ship systemd for user sessions
> though.

How far we are with this? If that will take a while perhaps we should
support the old D-Bus service too.


-- 
Luiz Augusto von Dentz

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

* Re: [PATCH] Allow using obexd without systemd in the user session
  2013-10-14 15:02     ` Luiz Augusto von Dentz
@ 2013-10-14 18:02       ` Bastien Nocera
  0 siblings, 0 replies; 6+ messages in thread
From: Bastien Nocera @ 2013-10-14 18:02 UTC (permalink / raw)
  To: Luiz Augusto von Dentz
  Cc: Marcel Holtmann, Giovanni Campagna,
	linux-bluetooth@vger.kernel.org, Giovanni Campagna

On Mon, 2013-10-14 at 18:02 +0300, Luiz Augusto von Dentz wrote:
> Hi Bastien,
> 
> On Sat, Oct 12, 2013 at 7:25 PM, Bastien Nocera <hadess@hadess.net> wrote:
> > On Sat, 2013-10-12 at 18:08 +0200, Marcel Holtmann wrote:
> >> Hi Giovanni,
> > <snip>
> >> that is something your distro specific package should be fixing if
> >> that is needed. The focus is on integration with systemd and nothing
> >> else. You can use --disable-systemd and provide your own set of D-Bus
> >> service files.
> >
> > There aren't any distributions that ship systemd for user sessions
> > though.
> 
> How far we are with this? If that will take a while perhaps we should
> support the old D-Bus service too.

It's planned for GNOME 3.12 for GNOME, so in about 6 months time
(April-ish release date).

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

end of thread, other threads:[~2013-10-14 18:02 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-10-12 15:45 [PATCH] Allow using obexd without systemd in the user session Giovanni Campagna
2013-10-12 16:08 ` Marcel Holtmann
2013-10-12 16:18   ` Giovanni Campagna
2013-10-12 16:25   ` Bastien Nocera
2013-10-14 15:02     ` Luiz Augusto von Dentz
2013-10-14 18:02       ` Bastien Nocera

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