Bug 29205

Summary: [PATCH] Please add systemd unit files
Product: udisks Reporter: Lennart Poettering <lennart>
Component: generalAssignee: David Zeuthen (not reading bugmail) <zeuthen>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: medium CC: systemd
Version: unspecified   
Hardware: Other   
OS: All   
Whiteboard:
i915 platform: i915 features:
Attachments: the patch
fix SystemdService= line in bus service file

Description Lennart Poettering 2010-07-21 17:22:59 UTC
See attachment for a patch that makes sure udisks installs systemd unit files by default.
Comment 1 Lennart Poettering 2010-07-21 17:23:24 UTC
Created attachment 37287 [details] [review]
the patch
Comment 2 Michael Biebl 2010-07-21 17:34:43 UTC
org.freedesktop.UDisks.service.in has
SystemdService=udisks.service

but you named the file udisks-daemon.service (fwiw I'd prefer the name udisks.service)

I'd also prefer if the service was started on demand only, and not automatically by graphical.target
Comment 3 Lennart Poettering 2010-07-21 18:11:45 UTC
Oh, thanks for the pointer. The reason I named it "udisks-daemon.service" is btw that the already is the avahi service file "udisks.service" in the data dir which means we cannot reuse the name.
Comment 4 Lennart Poettering 2010-07-21 18:13:11 UTC
Yepp, it kinda sucks that we now have three types of files with the .service suffix:

- D-Bus activation files
- systemd service files
- Avahi service description files

And what's even worse is that I am to blame for at least two of these.
Comment 5 Lennart Poettering 2010-07-21 18:15:46 UTC
Created attachment 37288 [details] [review]
fix SystemdService= line in bus service file
Comment 6 Michael Biebl 2010-07-21 19:58:10 UTC
(In reply to comment #3)
> Oh, thanks for the pointer. The reason I named it "udisks-daemon.service" is
> btw that the already is the avahi service file "udisks.service" in the data dir
> which means we cannot reuse the name.

Well, you could rename it to data/avahi/udisks.service, data/systemd/udisks.service and and probably also use data/dbus/..

Would actually be cleaner and less confusing imho.

But in the end, I'll leave it up to David
Comment 7 David Zeuthen (not reading bugmail) 2012-09-28 15:32:24 UTC
I added this (and other systemd goodies) to udisks 2.0 a while ago so closing.

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.