ci: github: Update for deprecation of add-path (backport some fixes to doc/zephyr.doxyfile.in) #33467
Conversation
|
Not sure I'm the best to approve this, I don't know this part. Is this a backport of an existing commit in master? Maybe add a reference to it? |
galak
changed the title
Github has deprecated add-path, so update the workflows that use it to the new method of setting the PATH. [Backport of fabf403] Signed-off-by: Kumar Gala <[email protected]>
Noticed those with most recent doxygen, we do not use those options, so it is safe to remove now. [Backport of f2b04d9] Signed-off-by: Anas Nashif <[email protected]> Signed-off-by: Kumar Gala <[email protected]>
I've been seeing these cause errors on the more recent versions of
Doxygen which come with Arch Linux for a while now. Fix these:
error: Illegal format for option TCL_SUBST, no equal sign ('=') specified for item 'YES'
$ZEPHYR_BASE/tests/kernel/mem_protect/futex/src/main.c:461: warning: end of file with unbalanced grouping commands
Just trying to get them out of my local output and as preparation for
whenever they start showing up for Ubuntu.
[Backport of ededeb2]
Signed-off-by: Martí Bolívar <[email protected]>
Add references in the commit messages. |
Builds of docs with doxygen 1.8.16 has a number of warnings of the form: 'warning: unbalanced grouping commands'. Fix those warnings be either balancing the group command or removing it. [Backport of a8171db] Signed-off-by: Kumar Gala <[email protected]>
|
@gmarull any thoughts on what's going on with the docs build issue with tests/kernel/common/src/intmath.c |
|
@galak I can't see anything suspicious on that file. Does it only happen on new Doxygen versions? |
No idea, whatever version of doxygen is getting used by the ci check. Looks like 1.8.17 |
|
maybe we should just exclude doxygen building tests in the branch now and get this going: diff --git a/doc/zephyr.doxyfile.in b/doc/zephyr.doxyfile.in
index 49d78b92cf..e63a8ae069 100644
--- a/doc/zephyr.doxyfile.in
+++ b/doc/zephyr.doxyfile.in
@@ -762,8 +762,7 @@ INPUT = @ZEPHYR_BASE@/include/ \
@ZEPHYR_BASE@/include/arch/nios2/ \
@ZEPHYR_BASE@/lib/libc/minimal/include/ \
@ZEPHYR_BASE@/include/net/dns_resolve.h \
- @ZEPHYR_BASE@/subsys/testsuite/ztest/include/ \
- @ZEPHYR_BASE@/tests/kernel/
+ @ZEPHYR_BASE@/subsys/testsuite/ztest/include/
|
No issue w/me, what implication does this have when we release the next v1.14.x version for docs on release? |
none, we do not link the tests in documentation, it is only available in doxygen output and was added just to make sure structure is correct. We can also fix that later, right now however this will unblock all those 1.14 PRs |
We are running into a doxygen error with intmath.c from tests/kernel/common/src/. intmath.c:29: warning: member with no name found. Drop tests/kernel from the doxygen input path for now. Signed-off-by: Kumar Gala <[email protected]>
Pushed that change into the PR. |
5 participants
Github has deprecated add-path, so update the workflows that use it to
the new method of setting the PATH.
Signed-off-by: Kumar Gala [email protected]