Skip to content

Conversation

@probonopd
Copy link
Contributor

@matthijskooijman
Copy link
Collaborator

Commit looks good to me. However, it seems that the trailing ; is not strictly required. The spec you linked says (emphasis by me):

Some keys can have multiple values. In such a case, the value of the key is specified as a plural: for example, string(s). The multiple values should be separated by a semicolon and the value of the key may be optionally terminated by a semicolon. Trailing empty strings must always be terminated with a semicolon. Semicolons in these values need to be escaped using ;.

@probonopd
Copy link
Contributor Author

probonopd commented Aug 20, 2016

At least desktop-file-validate complains when the trailing semicolon is missing,

@cmaglie cmaglie merged commit 66128da into arduino:master Aug 22, 2016
@cmaglie cmaglie added this to the Release 1.6.12 milestone Aug 22, 2016
@probonopd probonopd deleted the patch-1 branch August 22, 2016 14:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants