Bug 963899 - lightdm is misusing the preset file logic of systemd
Summary: lightdm is misusing the preset file logic of systemd
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: lightdm
Version: 19
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Rex Dieter
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-05-16 18:24 UTC by Lennart Poettering
Modified: 2013-10-06 02:22 UTC (History)
5 users (show)

Fixed In Version: lightdm-1.6.2-1.fc19
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-10-06 02:22:59 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)

Description Lennart Poettering 2013-05-16 18:24:18 UTC
This call tells me that lightdm.rpm is providing a preset file:

$ repoquery -q --qf="%{name}" --whatprovides '*/lib/systemd/system-preset/*.preset' | sort -u
...
lightdm
...

It shouldn't do that. Preset files are something that the distribution, spins of the distribution or administrators write. They policy that is applied to the OS, and should not be shipped by any normal RPMS. 

For Fedora, additions to the preset file must be kept in systemd.rpm and need to be accepted by FPC.

Please remove the preset file from the RPM. Thanks!

Comment 1 Fedora Update System 2013-05-21 02:05:16 UTC
lightdm-1.6.0-6.fc19 has been submitted as an update for Fedora 19.
https://admin.fedoraproject.org/updates/FEDORA-2013-8422/lightdm-1.6.0-6.fc19

Comment 2 Fedora Update System 2013-05-21 17:18:15 UTC
Package lightdm-1.6.0-6.fc19:
* should fix your issue,
* was pushed to the Fedora 19 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing lightdm-1.6.0-6.fc19'
as soon as you are able to.
Please go to the following url:
https://admin.fedoraproject.org/updates/FEDORA-2013-8422/lightdm-1.6.0-6.fc19
then log in and leave karma (feedback).

Comment 3 Adam Williamson 2013-05-22 00:57:27 UTC
I can't quite see why, but 'fixing' this seems to have broken default activation of lightdm. It's listed in /usr/lib/systemd/system-preset/90-display-manager.preset and the package scripts look to be in order, but if I build a MATE live image with 1.6.0-5 then it boots to MATE, but if I build one with 1.6.0-6 or 1.6.0-7, it boots to a console, and 'systemctl status lightdm.service' shows it disabled and not running. 'systemctl start lightdm.service' brings MATE up.

Comment 4 Rex Dieter 2013-05-22 03:29:10 UTC
I'll revert the change for this, until the issues with it can be sorted out.

Comment 5 Lennart Poettering 2013-06-06 10:35:48 UTC
Hmm, I guess 

https://bugzilla.redhat.com/show_bug.cgi?id=903690#c7

is the explanation here. Will fix this shortly.

Comment 6 Fedora Update System 2013-06-06 10:39:02 UTC
systemd-204-5.fc19 has been submitted as an update for Fedora 19.
https://admin.fedoraproject.org/updates/systemd-204-5.fc19

Comment 7 Rex Dieter 2013-06-06 11:58:32 UTC
I get "Access denied" trying to view bug #903690

Comment 8 Fedora Update System 2013-06-06 12:44:28 UTC
systemd-204-6.fc19 has been submitted as an update for Fedora 19.
https://admin.fedoraproject.org/updates/systemd-204-6.fc19

Comment 9 Fedora Update System 2013-06-06 17:30:29 UTC
Package systemd-204-6.fc19:
* should fix your issue,
* was pushed to the Fedora 19 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing systemd-204-6.fc19'
as soon as you are able to.
Please go to the following url:
https://admin.fedoraproject.org/updates/FEDORA-2013-10202/systemd-204-6.fc19
then log in and leave karma (feedback).

Comment 10 Adam Williamson 2013-06-06 19:46:53 UTC
rdieter: that comment doesn't look private, so I'll reproduce it here:

"If I understand presets correctly, any product specific preset can be named:

/lib/systemd/system-preset/<NUMBER < 90>-<PRODUCT>.preset

e.g.:

/lib/systemd/system-preset/80-RHEL-7-VARIANT-A.preset

as long as it has a catchall wildcard as the last line:
disable *

... which makes me question /lib/systemd/system-preset/90-display-manager.preset 

which should not have any effect due to 
disable *
in /lib/systemd/system-preset/90-default.preset"

Comment 11 Fedora Update System 2013-06-08 03:36:47 UTC
systemd-204-6.fc19 has been pushed to the Fedora 19 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 12 Adam Williamson 2013-06-08 04:43:10 UTC
Re-opening, because I think system 204-6 doesn't actually fix the bug that lightdm includes a preset file, right? It just makes it possible for us to take the preset out of lightdm package without breaking anything. IIUC. So now it's on Rex.

Comment 13 Rex Dieter 2013-10-06 02:22:59 UTC
* Tue Aug 20 2013 Rex Dieter <rdieter> 1.6.0-11
- remove systemd preset (#963899)


commit/fix got rolled into update
https://admin.fedoraproject.org/updates/FEDORA-2013-16388/lightdm-1.6.2-1.fc19


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