Bug 891452 - doxygen is missing dependencies for F18 texlive update
Summary: doxygen is missing dependencies for F18 texlive update
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: doxygen
Version: 18
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Rex Dieter
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-01-03 00:35 UTC by Benjamin Kosnik
Modified: 2013-08-09 05:51 UTC (History)
3 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2013-01-20 03:21:13 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)

Description Benjamin Kosnik 2013-01-03 00:35:50 UTC
Description of problem:

When using doxygen to create PDF output, the process errors out, with
messages like:

LaTeX error: file 'tocloft.sty' not found

Version-Release number of selected component (if applicable):

doxygen-1.8.1.2-1.fc18.x86_64

How reproducible:

I can reproduce this on F18, by building the gcc/libstdc++ documentation, configure and then "make pdf"

Steps to Reproduce:
1.
2.
3.
  
Actual results:


Expected results:


Additional info:

Adding these deps to the .spec file will solve the issue:


Requires:       texlive-sectsty
Requires:       texlive-tocloft
Requires:       texlive-xtab
Requires:       texlive-multirow

Comment 1 Rex Dieter 2013-01-03 04:32:13 UTC
I would argue it's the package(s) creating this documentation and using those newly split out texlive packages that need to include these additional BuildRequires

Comment 2 Rex Dieter 2013-01-03 04:35:29 UTC
Unless... are these texlive packages either specific to doxygen-created .pdf's or used by a majority or all of doxygen-created .pdf's?  If the answer is yes to either of those questions, then adding the deps here in doxygen would indeed be a convenience.

Comment 3 Rex Dieter 2013-01-03 04:42:30 UTC
Answering my own question, doxygen has never had any tex-related dependencies, so it's always been the responsibility of the documentation-building user or package to ensure the environment works.

Closing->NotABug

If you can identify any build failures caused by this issue, either file bugs against those particular components or mention them here, and I'll be happy to fix things up as needed.

Comment 4 Benjamin Kosnik 2013-01-03 17:37:03 UTC
The doxygen spec file here:

doxygen-1.8.2-1.fc19.src.rpm 

Seems incorrect to have:

BuildRequires: texlive-dvips
BuildRequires: texlive-utils
BuildRequires: texlive-latex

these are not build dependencies, (and texlive-utils no longer exists on F18), rather runtime and for pdflatex, and should be:

Requires: texlive-dvips
Requires: texlive-latex
Requires:       texlive-sectsty
Requires:       texlive-tocloft
Requires:       texlive-xtab
Requires:       texlive-multirow

Without these dependencies, doxygen will fail at PDF generation. I consider this a bug.

Comment 5 Benjamin Kosnik 2013-01-03 17:41:03 UTC
and the build failure is, I cannot "make pdf" in gcc without it. Thanks for your prompt attention.

Comment 6 Rex Dieter 2013-01-03 19:06:12 UTC
It is *gcc* documentation referencing those missing tex packages, not something doxygen should have to care about.  if one wants to build pdf docs from gcc.spec, one should add there the missing BuildRequires: texlive-...

The slippery slope here following your suggestion would be for doxygen to include *all* of texlive essentially to ensure that any/all .tex-based docs can build , which I hope you would agree would be counter-productive.

Comment 7 Benjamin Kosnik 2013-01-03 19:42:37 UTC
Ok. I actually built doxygen on F18. Here are the correct build deps:


BuildRequires: perl
BuildRequires: gettext
BuildRequires: flex
BuildRequires: bison
BuildRequires: desktop-file-utils
BuildRequires: texlive-latex
BuildRequires: gcc-c++
BuildRequires: texlive-epstopdf-bin
BuildRequires: ghostscript

Any use of doxygen to create PDF with pdflatex (USE_PDFLATEX = YES, which is the default for doxygen)  will also require the following runtime deps:

Requires: texlive-latex-bin-bin
Requires: texlive-sectsty
Requires: texlive-tocloft
Requires: texlive-xtab
Requires: texlive-multirow

Please fix the doxygen deps. This is a bug.

Any use of doxygen to create graphical heirarchy (HAVE_DOT = NO is the default for doxygen) will also require:

Requires: graphviz

Because dot is optional in the default doxygen-generated config file, I'm not insisting that it be added to the spec file. It's not an error to have it.

Comment 8 Rex Dieter 2013-01-03 19:53:26 UTC
OK, so now we're getting somewhere, providing a justification in response to my comment #2 .  thanks.

Comment 9 Rex Dieter 2013-01-03 20:09:16 UTC
%changelog
* Thu Jan 03 2013 Rex Dieter <rdieter> - 1:1.8.3-2
- doxygen is missing dependencies for texlive update (#891452)
...

Comment 10 Than Ngo 2013-01-03 20:19:24 UTC
it should be fixed in 1.8.3-2, Rex, thanks for taking care of the issue!

Comment 11 Fedora Update System 2013-01-03 20:32:42 UTC
doxygen-1.8.3-2.fc18 has been submitted as an update for Fedora 18.
https://admin.fedoraproject.org/updates/doxygen-1.8.3-2.fc18

Comment 12 Fedora Update System 2013-01-04 20:24:35 UTC
Package doxygen-1.8.3-2.fc18:
* should fix your issue,
* was pushed to the Fedora 18 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing doxygen-1.8.3-2.fc18'
as soon as you are able to.
Please go to the following url:
https://admin.fedoraproject.org/updates/FEDORA-2013-0169/doxygen-1.8.3-2.fc18
then log in and leave karma (feedback).

Comment 13 Fedora Update System 2013-01-09 22:53:26 UTC
Package doxygen-1.8.3-3.fc18:
* should fix your issue,
* was pushed to the Fedora 18 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing doxygen-1.8.3-3.fc18'
as soon as you are able to.
Please go to the following url:
https://admin.fedoraproject.org/updates/FEDORA-2013-0169/doxygen-1.8.3-3.fc18
then log in and leave karma (feedback).

Comment 14 Fedora Update System 2013-01-20 03:21:15 UTC
doxygen-1.8.3-3.fc18 has been pushed to the Fedora 18 stable repository.  If problems still persist, please make note of it in this bug report.


Note You need to log in before you can comment on or make changes to this bug.