https://bugzilla.redhat.com/show_bug.cgi?id=1324863 Bug ID: 1324863 Summary: Review Request: varnish-modules - A collection of modules extending varnish VCL Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nobody@xxxxxxxxxxxxxxxxx Reporter: ingvar@xxxxxxxxx QA Contact: extras-qa@xxxxxxxxxxxxxxxxx CC: package-review@xxxxxxxxxxxxxxxxxxxxxxx Spec URL: https://ingvar.fedorapeople.org/varnish/varnish-modules.spec SRPM URL: https://ingvar.fedorapeople.org/varnish/varnish-modules-0.9.0-1.fc23.src.rpm Description: This is a collection of modules ("vmods") extending Varnish VCL used for describing HTTP request/response policies with additional capabilities. This collection contains the following vmods (previously kept individually): cookie, vsthrottle, header, saintmode, softpurge, tcp, var, xkey Fedora Account System Username: ingvar copr builds for epel: https://copr.fedorainfracloud.org/coprs/ingvar/varnish41/ Review remarks predefended * License check lists some files lacking a proper license header. I have checked with upstream that all source files are BSD. I have requested that a license header is added to all source files. Upstream issue #9 * BuildRoot header is added. This is for epel backwards compatibility. * Package does run rm -rf %{buildroot}. This is for epel backwards compatibility. Ingvar -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx http://lists.fedoraproject.org/admin/lists/package-review@xxxxxxxxxxxxxxxxxxxxxxx
