Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add missing desktop entry keys to XDG autostart generator #17276

Closed
rydalt opened this issue Oct 7, 2020 · 3 comments
Closed

Add missing desktop entry keys to XDG autostart generator #17276

rydalt opened this issue Oct 7, 2020 · 3 comments

Comments

@rydalt
Copy link

rydalt commented Oct 7, 2020

Is your feature request related to a problem? Please describe.
Currently whenever I login I get a spew of log spam related to .desktop files and unknown keys from auto-start applications. Notably Path, MimeType, StartupWMClass, and URL are considered unknown keys, however each are standard keys per the Desktop Entry Specification and are not listed in xdg-autostart-service.c with the rest of the standard keys.

Here's an anonymized example from my logs:

MON DD HH:MM:SS hostname systemd[pid]: $HOME/.config/autostart/org.qbittorrent.qBittorrent.desktop:7: Unknown key name 'MimeType' in section 'Desktop Entry', ignoring.
MON DD HH:MM:SS hostname systemd[pid]: $HOME/.config/autostart/org.qbittorrent.qBittorrent.desktop:12: Unknown key name 'StartupWMClass' in section 'Desktop Entry', ignoring
MON DD HH:MM:SS hostname systemd[pid]: $HOME/.config/autostart/discord.desktop:10: Unknown key name 'Path' in section 'Desktop Entry', ignoring.

I don't have an example of the URL key being logged as unknown, but it appears to be missing in the code related to auto starting applications.

Describe the solution you'd like
Ideally each missing key would be added and would not be logged every time I login as a direct result.

Describe alternatives you've considered
I suppose they could just not be added and I can live with 5-8 lines of unnecessary log spam.

@mbiebl
Copy link
Contributor

mbiebl commented Oct 7, 2020

Maybe those log messages could be downgraded to debug (at least some of them).
Slightly related is this downstream bug report https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=968116

@rydalt
Copy link
Author

rydalt commented Oct 8, 2020 via email

@poettering
Copy link
Member

Yeah, we should downgade the messages at least for standard defined, but ignored keys.

/cc @benzea

benzea pushed a commit to benzea/systemd that referenced this issue Oct 8, 2020
It makes sense to ignore all the common fields that are expected and
that we can safely ignore. Note that it is fine to ignore URL as we will
already warn about the type= being wrong in that case.

Closes: systemd#17276
benzea pushed a commit to benzea/systemd that referenced this issue Oct 8, 2020
It makes sense to ignore all the common fields that are expected and
that we can safely ignore. Note that it is fine to ignore URL as we will
already warn about the type= being wrong in that case.

Closes: systemd#17276
benzea pushed a commit to benzea/systemd that referenced this issue Oct 8, 2020
It makes sense to ignore all the common fields that are expected and
that we can safely ignore. Note that it is fine to ignore URL as we will
already warn about the type= being wrong in that case.

Closes: systemd#17276
lqb pushed a commit to lqb/systemd that referenced this issue Oct 29, 2020
It makes sense to ignore all the common fields that are expected and
that we can safely ignore. Note that it is fine to ignore URL as we will
already warn about the type= being wrong in that case.

Closes: systemd#17276
mikhailnov pushed a commit to mikhailnov/systemd that referenced this issue Jan 16, 2021
It makes sense to ignore all the common fields that are expected and
that we can safely ignore. Note that it is fine to ignore URL as we will
already warn about the type= being wrong in that case.

Closes: systemd#17276
(cherry picked from commit 51ac77d)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

3 participants