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

Make repository REUSE compliant #97

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

xsuchy
Copy link

@xsuchy xsuchy commented Nov 21, 2023

@kraih
Copy link
Member

kraih commented Nov 21, 2023

Thanks, i'll have to read up on that spec. Not familiar with it yet.

@kraih
Copy link
Member

kraih commented Nov 23, 2023

Ignoring the question if we want to use the REUSE spec for Cavil. The license declarations are actually in large parts incorrect. All code and tests are GPL-2.0-or-later. Test data is entirely synthetic and has license declarations (often incorrect ones) merely for testing purposes. One exception being the included tarball of a Mojolicious Perl package release (which has its own set of licensing information) used for testing the unpacker and indexer.

@kraih
Copy link
Member

kraih commented Nov 23, 2023

Given that openSUSE has not adopted the REUSE spec in any form (i'm not aware it has ever even been discussed), i'm not sure why we would want this metadata in the repo tbh.

@kraih
Copy link
Member

kraih commented Nov 23, 2023

Of course i can see why this information would be attractive to packagers. But realistically, it's not something you're going to convince many upstream projects of maintaining for you long term. This is for Linux distributions to take care of themselves.

@xsuchy
Copy link
Author

xsuchy commented Nov 23, 2023

It is an interesting question how we should interpret SPDX-License-Identifier: MIT in these headers. I asked our legal for insights.
It would be likely better if there were comments that the SPDX-License-Identifier declaration there is just for testing purposes and not declaring actually license.

i'm not sure why we would want this metadata in the repo tbh.

The purpose of REUSE is that you can disambiguate the licenses in project. If... err... When more projects will use it, then projects like Cavil can stop guessing what license projects are using and use this curated data that can be trusted.
And maintenance of this data is not big pain. In case of Cavil close to zero.

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.

2 participants