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
comment:4 follow-up: 5 Changed 13 years ago by cssdev
Status: | new → assigned |
---|
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 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 follow-up: 8 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 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: | assigned → closed |
separate latex variant added in r84825
Due to Hurricane Irene, I won't be able to check things for a few more days.