Skip to content

Comments

ci/schemacheck.py: use BSD-3-Clause instead of MIT#237

Open
gagath wants to merge 1 commit intoqualcomm-linux:mainfrom
gagath:fix-schemacheck-license
Open

ci/schemacheck.py: use BSD-3-Clause instead of MIT#237
gagath wants to merge 1 commit intoqualcomm-linux:mainfrom
gagath:fix-schemacheck-license

Conversation

@gagath
Copy link
Contributor

@gagath gagath commented Feb 12, 2026

Running the reuse lint tool reports two different licenses being used in the project, while the README states that the whole project is BSD-3-Clause.

This file was added to the repository by a Qualcomm contributor, update its copyright notice and SPDX-License-Identifier to match the rest of the repository.

The spacing between the shebang and the license notice was automatically added by reuse annotate, so it is believed to be a good practise to split the two by an empty line.

Running the `reuse lint` tool reports two different licenses being used
in the project, while the README states that the whole project is
BSD-3-Clause.

This file was added to the repository by a Qualcomm contributor, update
its copyright notice and SPDX-License-Identifier to match the rest of
the repository.

The spacing between the shebang and the license notice was
automatically added by `reuse annotate`, so it is believed to be a good
practise to split the two by an empty line.

Signed-off-by: Agathe Porte <agathe.porte@oss.qualcomm.com>
@github-actions
Copy link

Test Results

 2 files  ±0   6 suites  ±0   2m 51s ⏱️ ±0s
20 tests ±0  20 ✅ ±0  0 💤 ±0  0 ❌ ±0 
64 runs  ±0  64 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 1cf5fe1. ± Comparison against base commit 709b643.

@github-actions
Copy link

Test jobs for commit 1cf5fe1

@lool
Copy link
Contributor

lool commented Feb 12, 2026

@gagath this file was contributed as part of the LAVA integration, and predates qcom-deb-images; I am not sure of its exact origin, but I'm sure @mwasilew would know, except he's OoO right now; so let's check with him when he's back, it's likely that this was developed by Foundries before its acquisition and that the license is indeed correct, not sure about the copyright though.

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