This is the mail archive of the cygwin-apps mailing list for the Cygwin project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [ITP] doxygen-1.8.0-2 -- A documentation system for C++, C, Java, Objective-C, IDL (Corba and Microsoft flavors) and to some extent PHP, C#, and D.

On 9/29/2012 1:21 PM, David Stacey wrote:

wget \ \ \ \

You should keep using -1 for subsequent build attempts until one gets RFU'd. Use new package versions only to disambiguate published package versions. The package version number is there to help setup.exe out, not to help us poor humans out. :)


1. Another build requirement change: the doxygen.README file still refers to TeTeX, which was replaced recently with TeX Live. But, installing a basic TeX setup alone isn't enough. To build the PDF manual, you need the optional texlive-collection-fontutils package for epstopdf, and texlive-collection-latexextra for float.sty. There's no need to mention any other packages. Installing those two into a stock Cygwin install will drag in enough of the remaining TeX stuff to build the manual.

2. /usr/share/doc/doxygen/latex should be removed, in favor of a .../pdf subdirectory. I don't see a reason to ship source and intermediate build files here, expecting the user to build the docs. What's wanted here is a PDF of the LaTeX *output*.

3. I'd rather see your first version be 1.8.2-1 instead. Why start with a version two bug fix releases back?

Nits aside, I can't withhold a GTG recommendation, since your packages are no worse off than mine were, and those were apparently still useful to people. :)

Something to pursue later, having no effect on GTG status:

There's a bogus test in Doxygen's configure script, where it goes looking for dot(1) from GraphViz. It does a weak check for it in a few common locations, and yells if it isn't there. dot(1) being a filter, there isn't a huge penalty for using the native Windows version, which of course doesn't get installed in any of those locations. It would be nice to either 1) send upstream a test using the PATH instead of a hardcoded list; or 2) adopt Yaakov's GraphViz package:

Option 1 is sufficient, but option 2 means you can make graphviz a dependency of your doxygen package, so everyone gets it automatically.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]