Hi, Ranjan Maitra via users wrote: > Btw, I noticed that 1.4.1 has the following as changelog: > > * Adjusted how/when the pseudo-root window is created and removed, in > preparation for handling multiple monitors in SLiM > * Adjusted slimlock to use the panel class in the way it was designed > to be used. > * Current version of SLiM fixed by allocating bimap always. > * Find '#' character > * That passwd_feedback wrong > * Changed the App::mcookiesize const member variable to a #define > * Reinstate install of systemd service file - required by Debian > * Note an expired passwords bug in the man page > * Ticket #3 : Use xinerama to pick a viewport (single monitor) for DM mode > too > * Fixed an annoying developer warning from cmake. > * Updated systemd service file to fix some debian / lintian messages. > > I do not know if the systemd service itself is > distribution-specific (perhaps so), but if not, would it > fix the problem? I don't know whether systemd even issues the warning for the slim.service as it doesn't mention /var/run there; it's only set in the config. The changes to use /run rather than /var/run may just be minor future-proofing to avoid problems if the /var/run symlink were removed. The Fedroa package ships its own slim.service unit file which has a few differences from upstream (which I presume are useful, but really don't know). Any improvements made to the upstream systemd unit file would have to be reviewed and merged into the unit file Fedora ships. The upstream changes to slim.service between 1.4.0 and 1.4.1 appear to be contained in this commit. https://sourceforge.net/p/slim-fork/code/60/tree/trunk/slim.service?diff=61f6a8bb7607298228d62bed:59 --- a/slim.service +++ b/slim.service @@ -1,10 +1,14 @@ [Unit] Description=SLiM Simple Login Manager +Documentation=man:slim(1) After=systemd-user-sessions.service [Service] ExecStart=/usr/bin/slim -n -s Restart=on-failure +CapabilityBoundingSet=~CAP_SYS_PTRACE + [Install] Alias=display-manager.service +WantedBy=multi-user.target Documentation=man:slim(1)' allows `systemctl status slim` to show a link to the man page. CapabilityBoundingSet=~CAP_SYS_PTRACE removes the services ability to trace arbitrary processes (refer to capabilities(7) for more details). It's not a bad thing to limit what the process can do. WantedBy=multi-user.target is likely not desireable as it would try to start slim in the multi-user target which might conflict with another desktop manager that is installed. It's probably better to leave that out to ensure that multiple desktop managers can be cleanly installed without fighting with each other. But that's just a cursory view. I am far from an expert on systemd units, particularly with respect to the graphical desktop setup. :) -- Todd
Attachment:
signature.asc
Description: PGP signature
-- _______________________________________________ users mailing list -- users@xxxxxxxxxxxxxxxxxxxxxxx To unsubscribe send an email to users-leave@xxxxxxxxxxxxxxxxxxxxxxx Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/users@xxxxxxxxxxxxxxxxxxxxxxx Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
