Skip to content

Key "MimeType" is a list and needs a semicolon as trailing character #5269

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

Merged
merged 1 commit into from
Aug 22, 2016
Merged

Key "MimeType" is a list and needs a semicolon as trailing character #5269

merged 1 commit into from
Aug 22, 2016

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