doxygen: Please add texlive-latex-extra dep (or possibly +latex variant)

[eborisch@…]

When using doxygen to create latex output, running make in the output dir hits a missing sectsty.sty error, in the same fashion as noted here: ​https://bugs.launchpad.net/ubuntu/+source/doxygen/+bug/701492

port provides /opt/local/share/texmf-texlive-dist/tex/latex/sectsty/sectsty.sty
/opt/local/share/texmf-texlive-dist/tex/latex/sectsty/sectsty.sty is provided by: texlive-latex-extra

Suggested fix

• Add texlive-latex-extra (provides share/texmf-texlive-dist/tex/latex/sectsty/sectsty.sty) as a dependency to doxygen
• Perhaps add a +latex variant that in turn requires texlive-latex-extra (so not everyone who wants HTML doxygen outputs needs to install all of texlive-bin -- a non-trivial install)

[css@…]

Due to Hurricane Irene, I won't be able to check things for a few more days.

[eborisch@…]

Too early to check on progress?

[css@…]

Sorry, I haven't moved on this due to limited Mac time. The docs variant includes latex support, so it should suffice to add texlive-latex-extra to that variant's dependencies.

[css@…]

Replying to css@…:

Sorry, I haven't moved on this due to limited Mac time. The docs variant includes latex support, so it should suffice to add texlive-latex-extra to that variant's dependencies.

... where it's already declared ...

Can you see if things work if you port install doxygen +docs?

[eborisch@…]

But the docs variant is to build doxygen's documentation; the variant description and name don't provide the user any indication that without it Doxygen will be unable to create pdf (via latex output) documentation. Someone familiar with doxygen may only want the output support, but not wait for the actual documentation to be re-created locally.

Perhaps something like this:

variant docs description {Include the doxygen PDF documentation and LaTeX} requires {latex} {
  build.target-append pdf
  use_parallel_build  no
}

variant latex description {Support latex/PDF Doxygen output} {
  destroot.target-append  install_docs
  depends_build-append  bin:pdflatex:texlive \
        bin:gs:ghostscript \
        port:texlive-latex-extra
}

[eborisch@…]

The destroot.target-append should have been in the docs variant:

variant docs description {Include the doxygen PDF documentation and LaTeX} requires {latex} {
  destroot.target-append  install_docs
  build.target-append     pdf
  use_parallel_build      no
}

variant latex description {Support latex/PDF Doxygen output} {
  depends_build-append  bin:pdflatex:texlive \
                        bin:gs:ghostscript \
                        port:texlive-latex-extra
}

[css@…]

Replying to eborisch@…:

But the docs variant is to build doxygen's documentation; the variant description and name don't provide the user any indication that without it Doxygen will be unable to create pdf (via latex output) documentation. Someone familiar with doxygen may only want the output support, but not wait for the actual documentation to be re-created locally.

I think that's a sensible suggestion. I'll try to patch it in as a new revision containing separate +docs and +latex variants.

[css@…]

separate latex variant added in r84825