Skip to content

doc/Makefile.in: Update Doxyfile to latest version before use#265

Merged
obgm merged 1 commit intoeclipse-tinydtls:mainfrom
mrdeep1:doxygen_update
Mar 18, 2026
Merged

doc/Makefile.in: Update Doxyfile to latest version before use#265
obgm merged 1 commit intoeclipse-tinydtls:mainfrom
mrdeep1:doxygen_update

Conversation

@mrdeep1
Copy link
Copy Markdown
Contributor

@mrdeep1 mrdeep1 commented Nov 20, 2025

Add in '$(DOXYGEN) -u' to update Doxyfile to the latest version to reduce any old configuration warning errors.

@mrdeep1
doc/Makefile.in: Update Doxyfile to latest version before use
5e4ea09 
Add in '$(DOXYGEN) -u' to update Doxyfile to the latest version to reduce
any old configuration warning errors.

Signed-off-by: Jon Shallow <supjps-libcoap@jpshallow.com>
@boaks
Copy link
Copy Markdown
Contributor

boaks commented Mar 18, 2026

@obgm

Any thoughts on this?

@mrdeep1
Copy link
Copy Markdown
Contributor Author

mrdeep1 commented Mar 18, 2026

It does not change any TinyDTLS code, but gets rid of a lot of build warnings.

@boaks
Copy link
Copy Markdown
Contributor

boaks commented Mar 18, 2026

I'm just not common with doxygen, so I'm not sure, if that causes also other issues.
What will happen, if the build is done in a not "connected environment"?

@mrdeep1
Copy link
Copy Markdown
Contributor Author

mrdeep1 commented Mar 18, 2026

All that happens is the current doxygen executable updates the doxygen configuration file in situ and has no need to reference any external information.

If there is any issue, this additional line would fail and the build would continue as if the line was not there.

@obgm
Copy link
Copy Markdown
Contributor

obgm commented Mar 18, 2026

I am not a big fan of this type of dynamic changes to the Doxyfile (that was auto-generated from Doxyfile.in). I would prefer to manually update the source, Doxyfile.in.

However, doxygen -u did never break anything for me, so running it at that point of the Makefile seems not that critical, and therefore, I do not object to applying this change.

obgm
obgm approved these changes Mar 18, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@obgm obgm left a comment

Choose a reason for hiding this comment

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

LGTM

@obgm obgm merged commit 194d827 into eclipse-tinydtls:main Mar 18, 2026
6 checks passed
@mrdeep1 mrdeep1 deleted the doxygen_update branch March 18, 2026 15:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@obgm obgm obgm approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@mrdeep1 @boaks @obgm