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

AND/OR not parsed correctly for metadata_license #181

Closed
jbicha opened this issue Mar 8, 2018 · 3 comments
Closed

AND/OR not parsed correctly for metadata_license #181

jbicha opened this issue Mar 8, 2018 · 3 comments

Comments

@jbicha
Copy link
Contributor

jbicha commented Mar 8, 2018

I guess I filed hughsie/appstream-glib#226 in the wrong place since my bug was specifically about appstreamcli.

I'll go ahead and copy what I posted there…

The aisleriot and gnome-terminal developer now uses this license in his AppStream metadata:

<metadata_license>GPL-3.0+ or GFDL-1.3-only</metadata_license>

GFDL-1.3-only is SPDX 3.0 syntax (see #180 )

But even if we change it to GPL-3.0+ or GFDL-1.3, appstreamcli validate complains that the metadata does not use a valid permissive license. For Debian and Ubuntu, I had to use a patch to drop the GPL-3.0+ or so that the app will show up in the GNOME Software app.

Substituting and doesn't work here either. or is the correct intent of the developer and should be sufficient for AppStream's concerns but and would not be acceptable here.

References

Appendix IV: SPDX License Expressions of https://spdx.org/specifications

@ximion
Copy link
Owner

ximion commented Mar 8, 2018

I think back then I intentionally did only allow SPDX license codes, and not license expressions for this tag, to keep the ambiguity low (and because I thought nobody would even bother to put an expression there.
The specification says at https://www.freedesktop.org/software/appstream/docs/chap-Metadata.html#sect-Metadata-GenericComponent

The license codes correspond to the identifiers found at the SPDX OpenSource License Registry.

Which admittedly isn't incredibly clear.
That being said, I am not very against supporting expressions there.
@hughsie what do you think, should we do that? (In any case I would clarify the wording in the spec)

OT: gnome-terminal as far as I remember, has always been an odd case, since AFAIR its maintainer refused to put the metadata under any license that was not the license of the software itself. Hence it was GPL only before and excluded from metadata collections.

@ximion
Copy link
Owner

ximion commented Mar 8, 2018

Ah, it looks like Richard already fixed this in his code. I will likely do the same in that case.

@jbicha
Copy link
Contributor Author

jbicha commented Mar 8, 2018

Yes, this was a recent breakthrough for gnome-terminal and a few projects under similar maintainership.

I've not seen this licensing anywhere else (well, it didn't work so that's one very good reason!)

@ximion ximion closed this as completed in 6195f8a Mar 25, 2018
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

No branches or pull requests

2 participants