[Bug 427481] Review Request: documentation-devil - Documentation tool chain

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: documentation-devil - Documentation tool chain


https://bugzilla.redhat.com/show_bug.cgi?id=427481





------- Additional Comments From petersen at redhat.com  2008-01-21 00:56 EST -------
As an experiment I tried the following:

$ rpm -q docbook-style-xsl
docbook-style-xsl-1.73.2-4.fc8.noarch
$ rm -rf documentation-devil-0.26
$ tar zxf documentation-devil-0.26.tgz
$ cd documentation-devil-0.26
$ rm -r xsl/docbook
$ sed -i -e
"s%http://docbook.sourceforge.net/release/xsl/1.72.0%http://docbook.sourceforge.net/release/xsl/1.73.2%"
xsl/*.xsl
$ make docs

and it seems to build ok.

So unless there are any known issues with docbook-xsl-1.73.2
I think it would be safe not to include "xsl/docbook/".
It would also reduce the installed size from 34MB to 14MB
and reduce the binary package from 7MB to 3MB.

Would it be possible upstream to change the license to be explicitly GPL2?
It would be good if there were explicit license headers in the source files.

In fop/ are both font-metrics/ and font-metrics-0.93/ needed?
In a way it would be nice if the font metric data could be generated
at build time but maybe that could be done later.

It might be better to move the desktop file to a separate file at some point
specially if it is going to be translated, or just to include it in the source?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.