Add doxygen settings to Mk/Uses

[dteske]

Summary

Some ports require doxygen to compile.
sysutils/cmocka for example complains about not having doxygen.

Test Plan

Add "USES=doxygen" to existing uses on sysutils/cmocka for testing.

[adamw]

I'm missing something. Are there multiple ports that provide doxygen binaries?

[bapt]

What is the point about adding this uses?

[bapt]

I mean it bring nothing that a regular BUILD_DEPENDS inside the port itself can achieve easily

[mat]

That is only one line of code worth. What problem are you trying to solve?

Mk/Uses/doxygen.mk

6

Why create a variable, it is only used once.

8

Should probably be:

BUILD_DEPENDS+= doxygen:${_DOXYGEN_PORT}

[dteske]

In D22628#495001, @mat wrote:

That is only one line of code worth. What problem are you trying to solve?

A lot of R-cran packages use doxygen. These R packages can sometimes be rather picky about the doxygen version, and so it becomes rather nice to be able to say USES=doxygen in the port Makefile and override _DOXYGEN_PORT in make.conf

Mk/Uses/doxygen.mk

6

To allow it to be overridden in make.conf

8

If that works, great, but I was just following the other similar settings in the file

[mat]

In D22628#495025, @dteske wrote:

A lot of R-cran packages use doxygen. These R packages can sometimes be rather picky about the doxygen version, and so it becomes rather nice to be able to say USES=doxygen in the port Makefile and override _DOXYGEN_PORT in make.conf

There is only one version of doxygen in the ports tree, and this seems to be specific R, so if something like this ever comes to live, it should probably happen within USES=cran.

Mk/Uses/doxygen.mk

6

But that variable is not overridable in make.conf.

8

Well, you were depending upon the existence of a file, not about the existence of a binary to run things. Unless the problem you are trying to solve requires the exitence of that file but does not need to execute it, you're ok. If you are trying to say "this ports needs to run doxygen", your version is wrong.

[dteske]

In D22628#495001, @mat wrote:

That is only one line of code worth. What problem are you trying to solve?

A lot of R-cran packages use doxygen. These R packages can sometimes be rather picky about the doxygen version, and so it becomes rather nice to be able to say USES=doxygen in the port Makefile and override _DOXYGEN_PORT in make.conf

In D22628#495048, @mat wrote:

In D22628#495025, @dteske wrote:

A lot of R-cran packages use doxygen. These R packages can sometimes be rather picky about the doxygen version, and so it becomes rather nice to be able to say USES=doxygen in the port Makefile and override _DOXYGEN_PORT in make.conf

There is only one version of doxygen in the ports tree, and this seems to be specific R, so if something like this ever comes to live, it should probably happen within USES=cran.

Only one version of doxygen at the head of the official ports tree, yes.

I do like your suggestion of handling this in cran when it is needed.

I can abandon this for now. Thanks for the review.

[dteske]

Mk/Uses/doxygen.mk

6

Ah, didn't know that. Thanks.