Should doxygen really be required?

[opm]

Should doxygen really be required?

[akva2]

No. Forgot to test without it i guess.

-------- Opprinnelig melding --------
Fra: Markus Blatt [email protected]
Dato: 16.02.2016 19:04 (GMT+01:00)
Til: OPM/opm-common [email protected]
Emne: [opm-common] Should doxygen really be required? (#75)

Just tested a new setup and cmake failed due to missing doxygen. Is that intended?

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/75.

[joakim-hove]

Everyone who has used the OPM doxygen documentation raise their hand.

< asbesto underwear on/>
16. feb. 2016 19:04 skrev "Markus Blatt" [email protected]
følgende:

Just tested a new setup and cmake failed due to missing doxygen. Is that
intended?

—
Reply to this email directly or view it on GitHub
#75.

[bska]

Everyone who has used the OPM doxygen documentation raise their hand.

Outside of its in-source C++ comment form?

[joakim-hove]

Yes
16. feb. 2016 20:12 skrev "Bård Skaflestad" [email protected]
følgende:

Everyone who has used the OPM doxygen documentation raise their hand.

Outside of its in-source C++ comment form?

—
Reply to this email directly or view it on GitHub
#75 (comment).

[bska]

Everyone who has used the OPM doxygen documentation raise their hand.

Outside of its in-source C++ comment form?

Yes

Well, I have looked at the formatted output occasionally, but mostly to verify that I didn't break it or to check the effects of certain formatting requests. I guess that technically counts as "using it", but I certainly don't look at it with any kind of regularity outside the comment form. That I do do however.

[atgeirr]

I agree that doxygen should not be required.

[andlaus]

Everyone who has used the OPM doxygen documentation raise their hand.

o/
/|
/ \

(at least for opm-material and ewoms.) but I think doxygen should only be an option, not a requirement.

[joakim-hove]

Hilarious ;-)
16. feb. 2016 22:44 skrev "Andreas Lauser" [email protected]
følgende:

Everyone who has used the OPM doxygen documentation raise their hand.

o/
/|
/ \

(at least for opm-material and ewoms.) but I think doxygen should only be
an option, not a requirement.

—
Reply to this email directly or view it on GitHub
#75 (comment).

[alfbr]

I have used it too. I don't think doxygen should be a build dependency. Whether we should ensure doxygen formatted comments/documentation is available is another discussion.

[akva2]

cannot reproduce. which unstable perturbation of the build system are you using?

[atgeirr]

Last message on this was "cannot reproduce" from @akva2, can you still produce the problem @blattms?

[pgdr]

Voting to close.

[joakim-hove]

Voting to close.

+1