Bug#968116: systemd: Not generating service for XDG autostart

[Michael Biebl]

Am 17.08.20 um 10:31 schrieb Francois Mescam:
> Le 16/08/2020 à 19:53, Michael Biebl a écrit :
>> Control: tags -1 + moreinfo
>>
>> Am 09.08.20 um 10:10 schrieb Francois Mescam:
>>> Package: systemd
>>> Version: 246-2
>>> Severity: normal
>>>
>>> Dear Maintainer,
>>>
>>> Since systemd 246-2 I have messages like these in the logs :
>>>
>>> Aug  9 08:46:18 eiffel7 systemd[24828]: Configuration file
>>> /etc/xdg/autostart/org.kde.discover.notifier.desktop is marked
>>> executable. Please remove executable permission bits. Proceeding
>>> +anyway.
>>> Aug  9 08:46:18 eiffel7 systemd[24828]:
>>> gnome-systemd-autostart-condition not found: No such file or directory
>>> Aug  9 08:46:18 eiffel7 systemd[24828]: kde-systemd-start-condition not
>>> found: No such file or directory
>>> Aug  9 08:46:18 eiffel7 systemd[24828]: Not generating service for XDG
>>> autostart app-at\x2dspi\x2ddbus\x2dbus-autostart.service, startup phases
>>> are not supported.
>>> Aug  9 08:46:18 eiffel7 systemd[24828]: Not generating service for XDG
>>> autostart app-gnome\x2dkeyring\x2dpkcs11-autostart.service, startup
>>> phases are not supported.
>>> Aug  9 08:46:18 eiffel7 systemd[24828]: Not generating service for XDG
>>> autostart app-gnome\x2dkeyring\x2dsecrets-autostart.service, startup
>>> phases are not supported.
>>> Aug  9 08:46:18 eiffel7 systemd[24828]: Not generating service for XDG
>>> autostart app-gnome\x2dkeyring\x2dssh-autostart.service, startup phases
>>> are not supported.
>>> Aug  9 08:46:18 eiffel7 systemd[24828]: Not generating service for XDG
>>> autostart app-powerdevil-autostart.service, only Type=Application is
>>> supported.
>>> Aug  9 08:46:18 eiffel7 systemd[24828]: Not generating service for XDG
>>> autostart app-pulseaudio-autostart.service, startup phases are not
>>> supported.
>>> Aug  9 08:46:18 eiffel7 systemd[24828]: Not generating service for XDG
>>> autostart app-xdg\x2duser\x2ddirs-autostart.service, startup phases are
>>> not supported.
>>> Aug  9 08:46:18 eiffel7 systemd[24828]: Not generating service for XDG
>>> autostart app-xfce4\x2dclipman\x2dplugin\x2dautostart-autostart.service,
>>> it is hidden.
>>> Aug  9 08:46:18 eiffel7 systemd[24828]: Not generating service for XDG
>>> autostart app-xfce4\x2dnotes\x2dautostart-autostart.service, it is
>>> hidden.
>>> Aug  9 08:46:18 eiffel7 systemd[24828]: Not generating service for XDG
>>> autostart app-xscreensaver-autostart.service, could not find TryExec=
>>> binary xscreensaver: No such file or directory
>>>
>>> I don't know if this is really a bug or if I have something to do to
>>> correct this question.
>> Not quite sure what this bug report is about, tbh.
>> Are you asking for these log messages to be downgraded to debug level so
>> you don't see them?
> Yes if these messages do not show something wrong which does not work
>> I don't really see the bug here, just the generator logging why it
>> doesn't generate service files for certain autostart files.
>> What is systemd package supposed to change here?
> 
> I don't know if not generating service files will have or not for
> consquence a malfunction.
> 
> So I am unable to say what is the good solution.

I'm undecided. I see your point that the XDG autostart generator is
maybe a bit chatty, but so are other programs if I look at journalctl
--user.
Most of those messages have log level info, so they do not indicate an
error.

> src/xdg-autostart-generator/xdg-autostart-service.c:                                return log_info_errno(r, "Exec binary '%s' does not exist: %m", c);
> src/xdg-autostart-generator/xdg-autostart-service.c:                log_info("Not generating service for XDG autostart %s, it is hidden.", service->name);
> src/xdg-autostart-generator/xdg-autostart-service.c:                log_info("Not generating service for XDG autostart %s, should be skipped by generator.", service->name);
> src/xdg-autostart-generator/xdg-autostart-service.c:                log_info("Not generating service for XDG autostart %s, only Type=Application is supported.", service->name);
> src/xdg-autostart-generator/xdg-autostart-service.c:                log_info("Not generating service for XDG autostart %s, startup phases are not supported.",


The following are warnings:
> src/xdg-autostart-generator/xdg-autostart-generator.c:                                log_warning_errno(errno, "Opening %s failed, ignoring: %m", *path);
> src/xdg-autostart-generator/xdg-autostart-generator.c:                FOREACH_DIRENT(de, d, log_warning_errno(errno, "Failed to enumerate directory %s, ignoring: %m", *path)) {
> src/xdg-autostart-generator/xdg-autostart-generator.c:                                log_warning_errno(errno, "stat() failed on %s/%s, ignoring: %m", *path, de->d_name);
> src/xdg-autostart-generator/xdg-autostart-service.c:                log_warning_errno(r, "Failed to parse %s, ignoring it", service->path);
> src/xdg-autostart-generator/xdg-autostart-service.c:                return log_warning_errno(SYNTHETIC_ERRNO(EINVAL), "Exec line is empty");
> src/xdg-autostart-generator/xdg-autostart-service.c:                log_warning("Not generating service for XDG autostart %s, it is has no Exec= line.", service->name);
> src/xdg-autostart-generator/xdg-autostart-service.c:                log_warning_errno(r,

And the following are errors:
> src/xdg-autostart-generator/xdg-autostart-condition.c:                return log_error_errno(SYNTHETIC_ERRNO(EINVAL),
> src/xdg-autostart-generator/xdg-autostart-service.c:                return log_error_errno(errno, "Failed to create unit file %s: %m", unit);


That looks somewhat sane to me.
If you think, those log messages need to be tweaked, or their priority,
please raise this upstream at https://github.com/systemd/systemd/issues

Regards,
Michael


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: OpenPGP digital signature
URL: <http://alioth-lists.debian.net/pipermail/pkg-systemd-maintainers/attachments/20200817/39f6e928/attachment.sig>