59 bytes, text/x-review-board-request
We should remove doxygen support from the build system. Right now, we have some very old code to support Doxygen in the tree: https://searchfox.org/mozilla-central/rev/e06af9c36a73a27864302cd2f829e6200dee8541/config/rules.mk#1551-1553 https://searchfox.org/mozilla-central/rev/e06af9c36a73a27864302cd2f829e6200dee8541/old-configure.in#3865-3889 It almost certainly doesn't work well. There's a ticket -- Bug 1150232 -- that tracks integrating Doxygen code comments into the |mach docs| documentation, but: - that ticket has gotten no traction - the engineer most interested in that work, Benoit Girard, is no longer contributing to the project - the gfx team is no longer using Doxygen for documentation (per kats in #developer IRC today): <nalexander> kats: do you know if the graphics team is still using Doxygen for documentation? <kats> nalexander: we aren't, no <nalexander> Nicholas Alexander kats: thanks! <kats> nalexander: it was always kind of ad-hoc to begin with We also have an _ancient_ ticket to update the configuration -- Bug 540381. I'm going to close those tickets in favour of this one. If we want Doxygen support in the future, we can work on integrating it into Sphinx in an appropriate way.
Assignee: nobody → nalexander
Status: NEW → ASSIGNED
Comment on attachment 8948008 [details] Bug 1435424 - Remove doxygen support from the build system. https://reviewboard.mozilla.org/r/217650/#review223454 Yeah, I'm pretty sure this is unused. If not, it's easy enough to recover it from version control :)
Attachment #8948008 - Flags: review?(gps) → review+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/63117daac3ab Remove doxygen support from the build system. r=gps
You need to log in before you can comment on or make changes to this bug.