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=809614

--- Comment #7 from adev <[email protected]> 2012-04-14 13:15:50 EDT ---
Updated from the comments :

Spec URL: http://firwen.org/home/specs/gfal2.spec
SRPM URL: http://firwen.org/home/specs/gfal2-2.0.0-0.6.beta.el5.centos.src.rpm

> If you're building the tar upstream from an evolving branch, consider adding a
> date, as in:
> http://fedoraproject.org/wiki/Packaging:SourceURL#Using_Revision_Control
> 
> I think it would be cleaner, it will help you debug later.

Thank you for the advice :)
I will act like this in my next reviews.

> You seem to have added for all but gfal2 and gfal2-transfer. Maybe add it 
> there
> too?

Corrected .

Regards,

Adrien

-- 
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.
_______________________________________________
package-review mailing list
[email protected]
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to