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-devel - Documentation tool chain https://bugzilla.redhat.com/show_bug.cgi?id=427481 petersen@xxxxxxxxxx changed: What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|nobody@xxxxxxxxxxxxxxxxx |petersen@xxxxxxxxxx Flag| |fedora-review? ------- Additional Comments From petersen@xxxxxxxxxx 2008-01-16 02:47 EST ------- Thanks for the update. :) Taking this review. Commenting below on http://svn.fedorahosted.org/svn/documentation-devel/trunk/Files/documentation-devil.spec since spec file in srpm appears to be slightly older. Just a few comments for upstream on the tarball: - Are all "Common_Content/common/*/images/*.png" the same? It would save quite a bit of space sharing the images (just a comment though). - It would be nice to preserve timestamps when generating the tarball if possible. - Upstream's docbook-xsl-1.72.0/doc/ is empty. - The latest upstream docbook-xsl is 1.73.2 any plans to update to it at some point? (docbook-style-xsl-1.73.2 is already in fedora) - There is po2xliff in translate-toolkit: dunno how it compares with po2xlf (these are not blockers - just things to consider upstream) -- 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. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review