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


Thomas Spura <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[email protected]
             Blocks|                            |177841(FE-NEEDSPONSOR)




--- Comment #2 from Thomas Spura <[email protected]>  2009-11-23 
15:55:42 EDT ---
Just a few comments, because I'm no sponsor anyway:

- You are not yet in the packager group and therefor you need a sponsor for
your fist package, see
http://fedoraproject.org/wiki/Package_Review_Process#Contributor

- Is this Review Requrest about togl17 or netgen?
  If you want to have both in fedora, you should have added 2 review requests.

- Could you upload the SPEC *and* SRPM to somewhere on the web and not just 
  pastebin?
  The first SPEC isn't available anymore...

- See https://fedoraproject.org/wiki/Packaging/Guidelines for details about how
  packaging should be done.

  e.g. to the togl17 spec: That package owns /* which is not what you want.
  The package must only own the directories it creates and the
subfiles/folders.
  Guessing in netgen you did the same.

-- 
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.

_______________________________________________
Fedora-package-review mailing list
[email protected]
http://www.redhat.com/mailman/listinfo/fedora-package-review

Reply via email to