touch-packages team mailing list archive

[Martin Pitt <martin.pitt@xxxxxxxxxx>]

+            xdiagnose (>= 3.7),

I don't want systemd to pull in xdiagnose. This will lead to pretty much
a disaster on servers :-) This should be left to seeds.

+-Requires=multi-user.target
++Requires=multi-user.target display-manager.service
+ After=multi-user.target
+ Conflicts=rescue.target
+ Wants=display-manager.service

This now seems redundant (Wants+Requires). I think I'm okay with this,
as having a graphical target without any DM seems a bit of a corner
case. I suppose this is necessary as otherwise OnFailure= wouldn't be
called for graphical.target if display-manager.service fails?

I wonder if we could make use of the new ".d/" support in 218 for this.
I. e. could the xdiagnose package just ship a /lib/systemd/system
/display-manager.service.d/xdiagnose.conf which adds the Requires= and
OnFailure=? Then we wouldn't need to teach systemd about this at all,
and xdiagnose would be self-contained.

-- 
You received this bug notification because you are a member of Ubuntu
Touch seeded packages, which is subscribed to systemd in Ubuntu.
https://bugs.launchpad.net/bugs/1400682

Title:
  Add xdiagnose fallback when display-manager fails to start

Status in systemd package in Ubuntu:
  Invalid

Bug description:
  Attached patch

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/systemd/+bug/1400682/+subscriptions