1435424 - Remove doxygen support from the build system

Status: RESOLVED FIXED

(Firefox Build System :: General, enhancement)

Description

[Nick Alexander :nalexander [he/him]]

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 1

[Nick Alexander :nalexander [he/him]]

Attachment: Bug 1435424 - Remove doxygen support from the build system.

This is ancient and the team that used it (gfx) is no longer using it.

Review commit: https://reviewboard.mozilla.org/r/217650/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/217650/

Comment 2

[Gregory Szorc [:gps]]

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 :)

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/63117daac3ab
Remove doxygen support from the build system. r=gps

Comment 4

[Noemi Erli[:noemi_erli]]

https://hg.mozilla.org/mozilla-central/rev/63117daac3ab