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

[minor] desktop file contains non-registered category #40

Closed
paulgevers opened this issue Jun 29, 2014 · 4 comments
Closed

[minor] desktop file contains non-registered category #40

paulgevers opened this issue Jun 29, 2014 · 4 comments
Assignees
Labels
Milestone

Comments

@paulgevers
Copy link
Contributor

lintian (a Debian package quality checking tool) reports that the "feed" category does not exist in the specification. I suggest to replace feed by news. You may even argue that also WebBrowser would make sense.

liferea: desktop-entry-invalid-category Feed usr/share/applications/liferea.desktop

This desktop entry lists a category that is not one of the registered
Main or Additional Categories in the FreeDesktop specification. Note
that case is significant and whitespace is only allowed immediately
before and after the equal sign in the Category key, not elsewhere in
the field.

The desktop-file-validate tool in the desktop-file-utils package is
useful for checking the syntax of desktop entries.

Refer to http://standards.freedesktop.org/menu-spec/1.0/apa.html for
details.

@lwindolf lwindolf added the bug label Jul 20, 2014
@lwindolf lwindolf added this to the 1.10.11 milestone Jul 20, 2014
@lwindolf lwindolf self-assigned this Jul 20, 2014
@mikelolasagasti
Copy link
Contributor

Version 1.0 of the spec doesn't cover the 'feed' category but version 1.1 does it.

http://standards.freedesktop.org/menu-spec/1.1/apas02.html

@lwindolf
Copy link
Owner

Development is on Ubuntu and "desktop-file-validate" returns no errors.

I know this is not very Debian-friendly, but given it is in 1.1 spec I'd say no bug and won't fix.

@paulgevers
Copy link
Contributor Author

Ok, agree, the lintian tool should be updated and you are still Debian-friendly ;). But, while we are at it, could you please add a "Version=1.1" line so that this is also clear. I wasn't aware that there was an update of the spec until you mentioned it in this bug report.

@lwindolf
Copy link
Owner

@paulgevers Done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants