Review Request: Fix the wrong usage of "Keywords" in one .dekstop file

Jekyll Wu adaptee at gmail.com
Fri Jul 13 14:49:03 UTC 2012


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/105555/
-----------------------------------------------------------

Review request for Plasma.


Description
-------

It is similar to review 105553.

According to the FDO specification[1][2], the type of "Keywords" is "localestring(s)", where multiple values are separated by a semicolon. 

The affected two .desktop files use comma instead of semicolon to separate multiple values. That seems to be the expected behavior for "X-KDE-Keywords", but not for "Keywords".

The patch simply uses semicolon and adds the trailing semicolon.


[1] http://standards.freedesktop.org/desktop-entry-spec/latest/ar01s05.html
[2] http://standards.freedesktop.org/desktop-entry-spec/latest/ar01s03.html


Diffs
-----

  applets/microblog/plasma-applet-microblog.desktop b8d3864 

Diff: http://git.reviewboard.kde.org/r/105555/diff/


Testing
-------


Thanks,

Jekyll Wu

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20120713/273001cf/attachment.html>


More information about the Plasma-devel mailing list