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

Warn about non SPDX licenses #1284

Open
xsuchy opened this issue Sep 20, 2024 · 3 comments
Open

Warn about non SPDX licenses #1284

xsuchy opened this issue Sep 20, 2024 · 3 comments

Comments

@xsuchy
Copy link
Member

xsuchy commented Sep 20, 2024

With most packages converted to SPDX, I propose that rpmlint (in F42) should warn when the package use Callaway syntax.

@danigm
Copy link
Member

danigm commented Oct 3, 2024

I suppose that is something to be done in the configuration file:
https://github.com/rpm-software-management/rpmlint/blob/main/configs/Fedora/licenses.toml

@xsuchy
Copy link
Member Author

xsuchy commented Oct 3, 2024

I think that rpmlint does not want to maintain a separate list of licenses anymore. This can be offloaded in runtime to data provided by fedora-license-data.

@hroncok
Copy link
Contributor

hroncok commented Oct 17, 2024

I suppose that is something to be done in the configuration file: https://github.com/rpm-software-management/rpmlint/blob/main/configs/Fedora/licenses.toml

We no longer use this file in Fedora. The Fedore config here is quite outdated.

I think that rpmlint does not want to maintain a separate list of licenses anymore. This can be offloaded in runtime to data provided by fedora-license-data.

In Fedora, I did that ~2 years ago: https://src.fedoraproject.org/rpms/rpmlint/pull-request/28

With most packages converted to SPDX, I propose that rpmlint (in F42) should warn when the package use Callaway syntax.

That is exactly1 what I did ~2 months ago in https://gitlab.com/fedora/legal/fedora-license-data/-/merge_requests/617

Footnotes

  1. Except it was for Fedora 41+, not 42+

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

3 participants