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


[EMAIL PROTECTED] changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|                            |fedora-review+




------- Additional Comments From [EMAIL PROTECTED]  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@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

Reply via email to