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 <lmacken@xxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Flag| |needinfo?(tla@xxxxxxxxx) --- Comment #9 from Luke Macken <lmacken@xxxxxxxxxx> 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@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review