https://bugzilla.redhat.com/show_bug.cgi?id=968947 --- Comment #2 from Ralph Bean <rbean@xxxxxxxxxx> --- Some preliminary comments: * The 'unzip' BuildRequires is not needed. See http://fedoraproject.org/wiki/Packaging/Guidelines#Exceptions_2 * There is bundling going on in lib/openid.php. Is that lib its own project? Should it be in a separate system package and symlinked in? At very least, it is a different license. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=tdTeL1uQDd&a=cc_unsubscribe _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review