Skip to content

Conversation

@jerpelea
Copy link
Contributor

Summary

According to the feedback from SPDX community we should use LicenseRef-NuttX-PublicDomain because NuttX-PublicDomain is not a valid SPDX id, so it will fail tests for SPDX spec compliance.

Impact

RELEASE

Testing

CI

According to the feedback from SPDX community we should use
LicenseRef-NuttX-PublicDomain because NuttX-PublicDomain
is not a valid SPDX id, so it will fail tests for SPDX spec compliance.

Signed-off-by: Alin Jerpelea <alin.jerpelea@sony.com>
@github-actions github-actions bot added Area: OS Components OS Components issues Area: Crypto Size: S The size of the change in this PR is small labels Dec 23, 2025
@jerpelea
Copy link
Contributor Author

@xiaoxiang781216 please merge

@acassis
Copy link
Contributor

acassis commented Dec 24, 2025

@jerpelea please fix these CI errors:

 Error: /home/runner/work/nuttx/nuttx/nuttx/crypto/rijndael.c:6:82: error: Long line found
Error: /home/runner/work/nuttx/nuttx/nuttx/include/crypto/rijndael.h:7:82: error: Long line found
Some checks failed. For contributing guidelines, see:
  https://github.com/apache/nuttx/blob/master/CONTRIBUTING.md
Error: Process completed with exit code 1.
``

@xiaoxiang781216
Copy link
Contributor

@xiaoxiang781216 please merge

but need fix the ci error first point out by @acassis .

@jerpelea
Copy link
Contributor Author

@xiaoxiang781216 please merge

but need fix the ci error first point out by @acassis .

unfortunately those errors ca not be fixed because they are used by the SPDX tools and must have that format

@jerpelea
Copy link
Contributor Author

@xiaoxiang781216 @acassis please note that this is a patch needed for the release

@xiaoxiang781216 xiaoxiang781216 merged commit 54b5a8f into apache:releases/12.12 Dec 26, 2025
36 of 40 checks passed
@jerpelea jerpelea deleted the bp-17645 branch December 26, 2025 16:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: Crypto Area: OS Components OS Components issues Size: S The size of the change in this PR is small

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants