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: squirrel - high level imperative/OO programming language https://bugzilla.redhat.com/show_bug.cgi?id=442280 j.w.r.degoede@xxxxxx changed: What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-review+ ------- Additional Comments From j.w.r.degoede@xxxxxx 2008-05-31 09:15 EST ------- Full review done: SHOULD FIX ---------- * Use standard (and actually working, unlike the current) sf.net download url of: http://downloads.sourceforge.net/squirrel/squirrel_%{version}.stable.tar.gz * You currently give README HISTORY and COPYRIGHT the timestamp of the COMPILE for for HISTORY this is different from the original timestamp of HISTORY and in future upstream releases chances are good the timestamps differ even more. Instead you should make backups while sed-ing using sed -i.orig and then use the .orig files as source for restoring the timestamps. No blockers, approved! -- 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