Skip to content

sys/include: use SPDX license tags [o-z]#22478

Open
kfessel wants to merge 1 commit into
RIOT-OS:masterfrom
kfessel:p-spdx-include-o-z
Open

sys/include: use SPDX license tags [o-z]#22478
kfessel wants to merge 1 commit into
RIOT-OS:masterfrom
kfessel:p-spdx-include-o-z

Conversation

@kfessel

@kfessel kfessel commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Contribution description

This took #21908 and has just [o-z] to avoid too many files Github issues (part 3/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:

  • none

@github-actions github-actions Bot added Area: timers Area: timer subsystems Area: SAUL Area: Sensor/Actuator Uber Layer Area: OTA Area: Over-the-air updates Area: USB Area: Universal Serial Bus Area: sys Area: System labels Jul 15, 2026
@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 14:19
@kfessel kfessel changed the title sys/include: use SPDX license tags[o-z] sys/include: use SPDX license tags [o-z] Jul 15, 2026
@crasbe crasbe added Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR CI: skip compile test If set, CI server will run only non-compile jobs, but no compile jobs or their dependent jobs labels Jul 15, 2026
@riot-ci

riot-ci commented Jul 15, 2026

Copy link
Copy Markdown

Murdock results

✔️ PASSED

0ffcafc sys/include: use SPDX license tags[o-z]

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

Artifacts

Comment thread sys/include/utlist.h
NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS
SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
*/
* SPDX-FileCopyrightText: 2007-2014 Troy D. Hanson http://troydhanson.github.com/uthash/

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
* SPDX-FileCopyrightText: 2007-2014 Troy D. Hanson http://troydhanson.github.com/uthash/
* SPDX-FileCopyrightText: 2007-2014 Troy D. Hanson
* SPDX-FileCopyrightText: http://troydhanson.github.com/uthash/

@kfessel kfessel Jul 16, 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.

wouldn't that be equal to

© 2007-2014 Troy D. Hanson © http://troydhanson.github.com/uthash/

vs

© 2007-2014 Troy D. Hanson http://troydhanson.github.com/uthash/

the github seems used like a email address in other statements

@kfessel kfessel Jul 16, 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.

but maybe we shall update the link

http://troydhanson.github.io/uthash/

or

https://github.com/troydhanson/uthash/

github.com is github only

github.io is for gh-pages

Suggested change
* SPDX-FileCopyrightText: 2007-2014 Troy D. Hanson http://troydhanson.github.com/uthash/
* SPDX-FileCopyrightText: 2007-2014 Troy D. Hanson http://troydhanson.github.io/uthash/

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.

upstream has updated it copyright info to github.io

and

also show that our import is outdated (not sure if there are security/safety issues) but that for another Time/PR

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

Labels

AI: None Stated AI was not (stated to be) used in this PR/Issue Area: OTA Area: Over-the-air updates Area: SAUL Area: Sensor/Actuator Uber Layer Area: sys Area: System Area: timers Area: timer subsystems Area: USB Area: Universal Serial Bus CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR CI: skip compile test If set, CI server will run only non-compile jobs, but no compile jobs or their dependent jobs 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