[Bug 529404] Review Request: cvc3 - Validity checker of many-sorted first-order formulas with theories
bugzilla at redhat.com
bugzilla at redhat.com
Mon Oct 19 22:29:13 UTC 2009
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=529404
--- Comment #6 from David A. Wheeler <dwheeler at dwheeler.com> 2009-10-19 18:29:12 EDT ---
I've looked over the new .spec file. Two small nits:
You'll need to submit the Java patch upstream, and document the date you did
so. The spec currently says:
# Fix Java build problems. This patch has not yet been sent upstream.
Patch3: cvc3-java.patch
I know you already knew that. :-)
I would recommend changing this:
rm -f doc/html/classHash_1_1hash__table__inherit__graph.png
to something like:
badfile="doc/html/classHash_1_1hash__table__inherit__graph.png"
if [ -f "$badfile" -a ! -s "$badfile" ] ; then
rm -f "$badfile"
fi
That way, if future doxygens are changed to handle this graph, re-building the
package will make it work automatically.
--
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.
More information about the Fedora-package-review
mailing list