Bug 1350750 - systemd units for lirc have errors
Summary: systemd units for lirc have errors
Alias: None
Product: Fedora
Classification: Fedora
Component: lirc
Version: 24
Hardware: x86_64
OS: Linux
Target Milestone: ---
Assignee: Alec Leamas
QA Contact: Fedora Extras Quality Assurance
Depends On:
TreeView+ depends on / blocked
Reported: 2016-06-28 09:40 UTC by Chris Warrick
Modified: 2016-07-02 15:24 UTC (History)
4 users (show)

Fixed In Version: lirc-0.9.4a-1.fc24
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Last Closed: 2016-07-02 15:24:30 UTC
Type: Bug

Attachments (Terms of Use)
systemd lirc unit bugs (9.73 KB, text/plain)
2016-06-28 09:40 UTC, Chris Warrick
no flags Details

Description Chris Warrick 2016-06-28 09:40:05 UTC
Created attachment 1173355 [details]
systemd lirc unit bugs

Description of problem:
systemd units for those units contain bugs that are not parsed correctly. Example:

Documentation=System daemon handling events from IR remotes decoded \
by lircd
;; broken into multiple lines, seemingly unsupported
; user =
: group =
;; first line is semicolon, second line is colon
ExecStart=/usr/sbin/irexec /etc/lirc/irexec.lircrc

Version-Release number of selected component (if applicable):

How reproducible:

Steps to Reproduce:
1. reload systemd units
2. wait a while
3. check journalctl

Actual results:
errors (see attachment)

Expected results:
correct load

Additional info:

Comment 1 Alec Leamas 2016-06-28 10:18:32 UTC
Hm... what's happening here?

- When I download a fresh lirc-core package using "dnf --releasever=24 download lirc-core" the downloaded lircd.service file is correctly commented with semicolons, not comma in both cases.
- Line continuation using \ is specified in the systemd.unit(5) manpage (and has been working well since long). Have you trailing whitespace after the  backslash.

Or, have you your own copies in /etc/systemd/system overriding the system defaults?

Comment 2 Chris Warrick 2016-06-28 10:30:32 UTC
irexec.service has the colon bug, and I can’t see trailing whitespace anywhere. Original copies.

Comment 3 Alec Leamas 2016-06-28 10:45:00 UTC
> irexec.service has the colon bug,

Indeed. Will fix.

>  I can’t see trailing whitespace anywhere.

Still, line continuation using \ is supported... What is the exact error you get?

Comment 4 Alec Leamas 2016-06-28 10:49:11 UTC
Ah... Documentation=  only allows URI:s, not free text. Need to fix this as well. Will update today or possibly tomorrow. I presume you can work around this for the moment.

Comment 5 Chris Warrick 2016-06-28 11:30:02 UTC
(I worked around it by removing lirc, which is completely unnecessary on a server. Located 1000+ km away from me. And in case you hadn’t noticed, there’s an attachment with all errors.)

Comment 6 Fedora Update System 2016-06-29 20:56:40 UTC
lirc-0.9.4a-1.fc24 has been submitted as an update to Fedora 24. https://bodhi.fedoraproject.org/updates/FEDORA-2016-90909780cf

Comment 7 Fedora Update System 2016-06-30 22:26:28 UTC
lirc-0.9.4a-1.fc24 has been pushed to the Fedora 24 testing repository. If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2016-90909780cf

Comment 8 Fedora Update System 2016-07-02 15:24:28 UTC
lirc-0.9.4a-1.fc24 has been pushed to the Fedora 24 stable repository. If problems still persist, please make note of it in this bug report.

Note You need to log in before you can comment on or make changes to this bug.