Remove doxygen support from the build system

RESOLVED FIXED in Firefox 60

Status

enhancement
RESOLVED FIXED
Last year
Last year

People

(Reporter: nalexander, Assigned: nalexander)

Tracking

unspecified
mozilla60

Firefox Tracking Flags

(firefox60 fixed)

Details

Attachments

(1 attachment)

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.
Comment hidden (mozreview-request)
Assignee: nobody → nalexander
Status: NEW → ASSIGNED

Comment 2

Last year
mozreview-review
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+

Comment 3

Last year
Pushed by nalexander@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/63117daac3ab
Remove doxygen support from the build system. r=gps

Comment 4

Last year
bugherder
https://hg.mozilla.org/mozilla-central/rev/63117daac3ab
Status: ASSIGNED → RESOLVED
Closed: Last year
Resolution: --- → FIXED
Target Milestone: --- → mozilla60

Updated

Last year
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.