Opened 13 years ago

Closed 13 years ago

#30988 closed defect (fixed)

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

Reported by: eborisch (Eric A. Borisch) Owned by: cssdev
Priority: Normal Milestone:
Component: ports Version: 2.0.1
Keywords: Cc:
Port: doxygen

Description

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)

Change History (9)

comment:1 Changed 13 years ago by ryandesign (Ryan Carsten Schmidt)

Cc: css@… removed
Owner: changed from macports-tickets@… to css@…

comment:2 Changed 13 years ago by cssdev

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

comment:3 Changed 13 years ago by eborisch (Eric A. Borisch)

Too early to check on progress?

comment:4 Changed 13 years ago by cssdev

Status: newassigned

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.

comment:5 in reply to:  4 Changed 13 years ago by cssdev

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?

comment:6 Changed 13 years ago by eborisch (Eric A. Borisch)

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
}

comment:7 Changed 13 years ago by eborisch (Eric A. Borisch)

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
}

comment:8 in reply to:  6 Changed 13 years ago by cssdev

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.

comment:9 Changed 13 years ago by cssdev

Resolution: fixed
Status: assignedclosed

separate latex variant added in r84825

Note: See TracTickets for help on using tickets.