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

Ken Dreyer <ktdre...@ktdreyer.com> changed:

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

--- Comment #6 from Ken Dreyer <ktdre...@ktdreyer.com> 2012-03-22 00:30:28 EDT 
---
One last, minor thing: I recommend dropping dist.ini from your package, since
it's of little value to users. Don't sweat re-rolling an SRPM just for this;
you can take care of it when you're importing it into Fedora.

EPEL doesn't currently have all the dependencies necessary, so it will more
work there (see eg.
http://koji.fedoraproject.org/koji/taskinfo?taskID=3921241).

Thanks for addressing all the concerns in the review. This package is approved.
What's your FAS username? I'll open a ticket with FESCo to get you sponsored
into the packager group.

-- 
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
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to