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

LICENSE

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

@xiaoxiang781216
Copy link
Contributor

@xiaoxiang781216 please merge

need fix the warning:

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
``

@jerpelea
Copy link
Contributor Author

nuttx/crypto/rijndael.c

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

@xiaoxiang781216 xiaoxiang781216 merged commit 4a06935 into apache:master Dec 26, 2025
59 of 77 checks passed
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.

3 participants