Skip to content

sys/include: use SPDX license tags [a-m]#22476

Open
kfessel wants to merge 3 commits into
RIOT-OS:masterfrom
kfessel:p-spdx-include-a-m
Open

sys/include: use SPDX license tags [a-m]#22476
kfessel wants to merge 3 commits into
RIOT-OS:masterfrom
kfessel:p-spdx-include-a-m

Conversation

@kfessel

@kfessel kfessel commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Contribution description

This took #21908 and has just [a-m] (mostly 'net') to avoid too many files Github issues (part 1/3)

#22475 got Licenses and may be (or not) merged first

Testing procedure

check diff for correctness
@maribu did when creating the original @kfessel does while creating this before switching it from draft

Issues/PRs references

#21515 tracking issue
#22476 [a-m]
#22477 [n]
#22478 [o-z]
#21908 original
#22475 Licenses
#22324 another one has given up

Declaration of AI-Tools / LLMs usage:

AI-Tools / LLMs that were used are:

  • some M365copilot chat was used to identify the ICU license (it actually suggested it might be X11, ICU was found by @kfessel)
  • AI/LLM did no edits and nothing was copied from an LLM interaction
  • none otherwise

@kfessel kfessel left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

MIT modern isn't the same as MIT

knapp daneben is trotzdem vorbei

it is MIT but with something about advertisement so little like MIT-modern or MIT-open-group but more like X11 a github emUnit actually says its (MIT/X Consotium) but with something about documentation -> its is "SPDX-License-Identifier: ICU"

https://spdx.org/licenses/ICU.htm

* CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF OR IN
* CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
*
* Except as contained in this notice, the name of a copyright holder

@kfessel kfessel Jul 15, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

https://spdx.org/licenses/MIT-open-group.html
MIT-open-group

 * SPDX-License-Identifier: MIT-open-group

@kfessel kfessel Jul 15, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

no its not it is ICU

its an variation of X11 which is a variation of MIT

and that is identified by ICU

Why ,Why, Why

Comment thread sys/include/embUnit/AssertImpl.h Outdated
Comment thread sys/include/embUnit/embUnit.h Outdated
Comment thread sys/include/embUnit/embUnit_config.h Outdated
Comment thread sys/include/embUnit/HelperMacro.h Outdated
Comment thread sys/include/embUnit/TestRunner.h Outdated
Comment thread sys/include/embUnit/TestSuite.h Outdated
Comment thread sys/include/embUnit/TextOutputter.h Outdated
Comment thread sys/include/embUnit/TextUIRunner.h Outdated
Comment thread sys/include/embUnit/XMLOutputter.h Outdated
Comment thread sys/include/embUnit/CompilerOutputter.h Outdated
@AnnsAnns AnnsAnns added the AI: None Stated AI was not (stated to be) used in this PR/Issue label Jul 15, 2026
@kfessel kfessel marked this pull request as ready for review July 15, 2026 13:55
@kfessel kfessel requested review from AnnsAnns, crasbe and maribu July 15, 2026 13:56
@kfessel

kfessel commented Jul 15, 2026

Copy link
Copy Markdown
Contributor Author

@AnnsAnns please reevalute the Ai tag

@AnnsAnns

Copy link
Copy Markdown
Member

ty for pointing that out

@AnnsAnns AnnsAnns added AI: Ducky PR/Issue uses AI for asking questions, AI was not involved in final product. CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR and removed AI: None Stated AI was not (stated to be) used in this PR/Issue labels Jul 15, 2026
@riot-ci

riot-ci commented Jul 15, 2026

Copy link
Copy Markdown

Murdock results

✔️ PASSED

531709d f: Apply suggestion from @kfessel

Success Failures Total Runtime
1 0 1 01m:27s

Artifacts

@crasbe crasbe added the Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation label Jul 15, 2026
Comment thread sys/include/hashes/sha3.h Outdated
@kfessel kfessel force-pushed the p-spdx-include-a-m branch from 70e3d90 to 531709d Compare July 16, 2026 12:44
@kfessel

kfessel commented Jul 16, 2026

Copy link
Copy Markdown
Contributor Author

rebased for xipfs

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AI: Ducky PR/Issue uses AI for asking questions, AI was not involved in final product. Area: sys Area: System Area: timers Area: timer subsystems CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants