Skip to content

Update Doxygen config#142

Merged
joto merged 1 commit into
mapbox:masterfrom
joto:doxygen-update
May 22, 2026
Merged

Update Doxygen config#142
joto merged 1 commit into
mapbox:masterfrom
joto:doxygen-update

Conversation

@joto
Copy link
Copy Markdown
Collaborator

@joto joto commented May 21, 2026

Using "doxygen -u"

@joto joto force-pushed the doxygen-update branch from acdcda2 to 8d826c5 Compare May 21, 2026 14:55
@kkaefer
Copy link
Copy Markdown
Member

kkaefer commented May 21, 2026

While looking through this, I noticed that the readme/contributing/documentation files reference a Makefile that no longer exists. Apparently, you deleted it in 2017 in 178c469 :)

@joto
Copy link
Copy Markdown
Collaborator Author

joto commented May 21, 2026

@kkaefer I can't find any mentioning of a Makefile!?

Using "doxygen -u"
@joto joto force-pushed the doxygen-update branch from 8d826c5 to d61fe9f Compare May 21, 2026 18:30
@kkaefer
Copy link
Copy Markdown
Member

kkaefer commented May 22, 2026

Not in the doxygen config itself, but in the rest of the readme files in our repo.

@joto
Copy link
Copy Markdown
Collaborator Author

joto commented May 22, 2026

Not in the doxygen config itself, but in the rest of the readme files in our repo.

I can't find anything there either.

@kkaefer
Copy link
Copy Markdown
Member

kkaefer commented May 22, 2026

I can :) See #147

@joto
Copy link
Copy Markdown
Collaborator Author

joto commented May 22, 2026

Those are not "left-overs" from before. They are just what I type in to get the job done. I don't use those cmake commands and I don't know why I should. Sure, now you can use ninja or whatever instead of make. But in the end cmake is calling the underlying tool. And I'd rather use the underlying tool directly, because cmake is hiding functionality and I know how to get the result I want with the underlying tool. Maybe I am old-fashioned.

@joto joto merged commit 203aa15 into mapbox:master May 22, 2026
31 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants