Skip to content

Use SPDX license #262

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

Conversation

StykMartin
Copy link
Contributor

@StykMartin StykMartin commented May 26, 2025

The Fedora Project is working to switch all licenses to SPDX format. When they converted our project, they listed the Callaway license for BSD.

I looked into why this was included and found that Dan C explained it when he added it. The file LabController/src/bkr/labcontroller/tback.py was originally from Django and used the 3-clause BSD license, but we have since removed it.

This pull request simply aligns the license with the one we have in Fedora.

The following document may provide additional answers:
https://fedoraproject.org/wiki/Changes/SPDX_Licenses_Phase_1

The file LabController/src/bkr/labcontroller/tback.py, originally from Django
and licensed under the 3-clause BSD license, was removed.

Signed-off-by: Martin Styk <[email protected]>
@StykMartin StykMartin requested a review from a team May 26, 2025 16:22
@StykMartin StykMartin self-assigned this May 26, 2025
@StykMartin StykMartin requested review from JohnVillalovos, p3ck and mdujava and removed request for a team May 26, 2025 16:22
Copy link
Collaborator

@JohnVillalovos JohnVillalovos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks @StykMartin

@StykMartin StykMartin merged commit 4bccb77 into beaker-project:release-29 May 27, 2025
15 checks passed
@StykMartin StykMartin deleted the wip/mstyk/use-spdx-license branch May 27, 2025 09:51
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