sys/include: use SPDX license tags [n]#22477
Conversation
github-actions
Bot
added
Area: network
AnnsAnns
added
the
AI: None Stated
| * we set our DTLS endpoint to use DTLS version 1.2 and act as a DTLS server. | ||
| * | ||
| * Note that some DTLS implementation do not support earlier versions of DTLS. | ||
| * In this case, @ref sock_dtls_create() will return an error. A list of |
There was a problem hiding this comment.
@AnnsAnns are you able to guess why these reference errors happen?
other references (same file work fine)
There was a problem hiding this comment.
I had good luck with removing the () AFAIR.
|
i published this PR even though there are reference errors with doxygen since the actual changes are probably unrelated |
crasbe
added
Type: cleanup
| * we set our DTLS endpoint to use DTLS version 1.2 and act as a DTLS server. | ||
| * | ||
| * Note that some DTLS implementation do not support earlier versions of DTLS. | ||
| * In this case, @ref sock_dtls_create() will return an error. A list of |
There was a problem hiding this comment.
I had good luck with removing the () AFAIR.
| * This file is subject to the terms and conditions of the GNU Lesser | ||
| * General Public License v2.1. See the file LICENSE in the top level | ||
| * directory for more details. | ||
| * SPDX-FileCopyrightText: 2015-2017 Ken Bannister. All rights reserved. |
There was a problem hiding this comment.
| * SPDX-FileCopyrightText: 2015-2017 Ken Bannister. All rights reserved. | |
| * SPDX-FileCopyrightText: 2015-2017 Ken Bannister |
Vermutlich nicht notwendig.
There was a problem hiding this comment.
i'd like to not change that
Co-authored-by: crasbe <crasbe@gmail.com>
did not work this time and there are more of these reference issues the static test is pointing to (no ()) shall i just add the filter? |
|
ahh i see that doxygen f-up was there before and also before it was just painted over dist/tools/doccheck/exclude_simple so i just need to paint over again for this pr seems like i need to update #22327 |
kfessel
force-pushed
the
p-spdx-include-n
branch
from
729c115 to
b1eb66d
Compare
| boards/msbiot/doc.md:73: warning: unable to resolve reference to 'LED1_ON' for \ref command | ||
| boards/msbiot/doc.md:73: warning: unable to resolve reference to 'LED1_OFF' for \ref command | ||
| boards/msbiot/doc.md:73: warning: unable to resolve reference to 'LED1_TOGGLE' for \ref command | ||
| sys/include/net/gnrc/netif/pktq/type.h:12: warning: Potential recursion while resolving \ref command! |
There was a problem hiding this comment.
this line moved up an became L9
at fist i though this was a doublet
4 participants
Contribution description
This took #21908 and has just [n] (mostly 'net') to avoid too many files Github issues (part 2/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: