Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=953514

Kevin Fenzi <ke...@scrye.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |ke...@scrye.com

--- Comment #1 from Kevin Fenzi <ke...@scrye.com> ---
So, some general comments. ;) 

- Usually when we package extensions or plugins for a package they have the
base package name in the name of the extension or plugin. So, you might
consider renaming this: 
varnish-extension-querystring or varnish-vmod-querystring?

- You cannot do conditional builds like that in the Fedora buildsystem. I'd
suggest doing a 'BuildRequires: varnish-exact-version' and set it to that
version. Then, when varnish is updated, you update that to the new version and
rebuild. 

- This needs patches to varnish? Have you opened a bug to get the Fedora
varnish maintainer(s) feedback on applying those?

-- 
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=KVNRSLGxQF&a=cc_unsubscribe
_______________________________________________
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to