Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 508450 - net-irc/quassel - add systemd service for core
Summary: net-irc/quassel - add systemd service for core
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Patrick Lauer
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: install-systemd-unit
  Show dependency tree
 
Reported: 2014-04-22 19:24 UTC by Konstantin (elxa)
Modified: 2014-05-03 18:43 UTC (History)
5 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
systemd service unit for quassel core (quassel.service,311 bytes, text/plain)
2014-04-22 19:28 UTC, Konstantin (elxa)
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Konstantin (elxa) 2014-04-22 19:24:19 UTC
The core component of quassel only ships the init script for openrc. It should include a systemd service, too.

Reproducible: Always

Steps to Reproduce:
emerge quassel with core enabled and try to start it with systemd as your init system.
Actual Results:  
no service for quassel available

Expected Results:  
service for systemd shipped with quassel ebuild
Comment 1 Konstantin (elxa) 2014-04-22 19:28:19 UTC
Created attachment 375498 [details]
systemd service unit for quassel core

Thanks to thomasg from #gentoo.de for providing this service file !

It currently depends on postgresql, because it is one of (three?) possible sql backends (sqlite, mysql, postgresql) for quassel core. Maybe gentoo should ship a service template and set the appropriate dependencies in the service file according to the USE of quassel. Modify it as needed for you sql backend.
Comment 2 Konstantin (elxa) 2014-04-23 15:32:01 UTC
Actually there are some errors in that service file. I will upload a better one in a few days. I don't have enough time to test this right now.
Comment 3 Mike Gilbert gentoo-dev 2014-04-23 20:01:18 UTC
Drop the EnvironmentFile and related variables. Also drop the postgres stuff.

If the sysadmin wants to change those values or use postgres, he is going to have to tweak the unit file by copying it to /etc/systemd/system or overriding values via /etc/systemd/system/quassel.service.d.
Comment 4 Mike Gilbert gentoo-dev 2014-04-23 20:04:25 UTC
Also, you don't need PIDFile for a non-forking service.
Comment 5 Pacho Ramos gentoo-dev 2014-05-02 10:28:44 UTC
+*quassel-0.10.0-r1 (02 May 2014)
+
+  02 May 2014; Pacho Ramos <pacho@gentoo.org> +files/quasselcore.service,
+  +quassel-0.10.0-r1.ebuild:
+  Add unit file (#508450 by Konstantin)
+
Comment 6 Johannes Huber (RETIRED) gentoo-dev 2014-05-03 18:43:06 UTC
Just for the record.

+
+  03 May 2014; Johannes Huber <johu@gentoo.org> -quassel-0.10.0.ebuild,
+  quassel-9999.ebuild:
+  Add systemd unit file to live ebuild as well, bug #508450. Remove old.
+