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=674692 Luke Macken <lmac...@redhat.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Flag| |needinfo?(t...@rasmil.dk) --- Comment #9 from Luke Macken <lmac...@redhat.com> 2011-02-24 12:53:22 EST --- (In reply to comment #8) > Package Review > ============== > [!] : MUST - All build dependencies are listed in BuildRequires, except for > any > that are listed in the exceptions section > of Packaging Guidelines. > BuildRequires: python2-devel is missing > http://fedoraproject.org/wiki/Packaging:Python#BuildRequires Added. > [?] : SHOULD - Package functions as described. You can test that the package works as expected by doing the following: $ paster create -t pyramid_alchemy testpyramid $ cd testpyramid $ paster serve development.ini Then fire up your web browser and make sure the default page loads. > [!] : SHOULD - Patches link to upstream bugs/comments/lists or are otherwise > justified. > OK, but an comment about the bug, would be nice. > http://fedoraproject.org/wiki/PackagingGuidelines#All_patches_should_have_an_upstream_bug_link_or_comment Comment added. Spec and SRPM updated, with the same URL as above. -- 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