Uh oh!
There was an error while loading. Please reload this page.
[SKIP CI] .github: doxygen: upgrade to Ubuntu 22.04 - #5979
Merged
Conversation
This should upgrade doxygen from version 1.8.17 to 1.9.1 This will hopefully provide FAIL_ON_WARNINGS and fix issues like doxygen/doxygen#7970 Signed-off-by: Marc Herbert <marc.herbert@intel.com>
CollaboratorAuthor
https://github.com/thesofproject/sof/runs/7157184365 fails with: This warning was escaping WARN_AS_ERROR because of doxygen 1.8.17 bug. It was printed but not changing the exist status. doxygen 1.9.1 in Ubuntu 22 does not miss it. So this PR is blocked until this warning is fixed, @singalsu is working on the warning. |
Unlike the older 'YES', FAIL_ON_WARNINGS reports all warnings. Signed-off-by: Marc Herbert <marc.herbert@intel.com>
lgirdwood
approved these changes
Jul 4, 2022
Fixes doxygen warning: warning: Include graph for 'src_ipc4_int32_table.h' not generated, too many nodes (89), threshold is 50. Consider increasing DOT_GRAPH_MAX_NODES. This code is generated and pollutes doxygen output anyway. Temporary, manual solution until the `\cond` directive is also generated by the sof/tools/tune/src/*export*.m script(s) Signed-off-by: Marc Herbert <marc.herbert@intel.com>
Fixes doxygen warning: warning: Included by graph for 'src.h' not generated, too many nodes (111), threshold is 50. Consider increasing DOT_GRAPH_MAX_NODES. This code is generated and pollutes doxygen output anyway. Temporary, manual solution until the `\cond` directive is also generated by the sof/tools/tune/src/*export*.m script(s) Signed-off-by: Marc Herbert <marc.herbert@intel.com>
CollaboratorAuthor
@singalsu is on vacation so went ahead and I manually added This fixes the doxygen build in https://github.com/thesofproject/sof/runs/7224393035, even with Ubuntu 22.04 The fix for the other, unrelated warnings in the new "sof-docs" build below has been submitted in |
marc-hb
marked this pull request as ready for review
July 6, 2022 23:29
marc-hb
requested review from
aborisovich, cujomalainey, dbaluta, lbetlej, mmaka1 and plbossart
as code ownersJuly 6, 2022 23:29
lgirdwood
approved these changes
Jul 7, 2022
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This upgrades doxygen from version 1.8.17 to 1.9.1
This provides FAIL_ON_WARNINGS and fixes issues like
doxygen/doxygen#7970
Signed-off-by: Marc Herbert marc.herbert@intel.com