Summary: | www/mod_security3: Move to security/modsecurity3 | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | Ports & Packages | Reporter: | Marie Helene Kvello-Aune <freebsd> | ||||||||
Component: | Individual Port(s) | Assignee: | Dag-Erling Smørgrav <des> | ||||||||
Status: | Closed FIXED | ||||||||||
Severity: | Affects Only Me | CC: | brnrd, des, joneum, marius.halden | ||||||||
Priority: | --- | Keywords: | easy, needs-qa | ||||||||
Version: | Latest | Flags: | marius.halden:
maintainer-feedback+
|
||||||||
Hardware: | Any | ||||||||||
OS: | Any | ||||||||||
See Also: | https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=227917 | ||||||||||
Attachments: |
|
Description
Marie Helene Kvello-Aune
2018-11-20 10:49:03 UTC
Maintainer OK Nice patch Marie. For the MOVED comment, id go for something more explicit/absolute, like: Not an apache module, use correct upstream name Hi Marie, thanks for the PR. I also wanted to work on this port today for NGINX and Apache. There are already version 3.0.3. Why do you want to move the port to security? mod_security is also in www Created attachment 199375 [details] rename of modsecurity3 (patch-compatible) ftr: This patch builds on Marius Halden's previous work on getting this done. Suggested commit message: Move www/mod_security3 to security/modsecurity3 because it's not an Apache module, and to reflect upstreams naming. PR: https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=233351 Sponsored by: Modirum (In reply to Jochen Neumeister from comment #3) I'm moving it from www/mod_security3 because it's not an Apache module, but a security-related module. I don't know if this holds true for www/mod_security. See comment by Bernard Spil (wearing apache@ hat) in comment 1 of bug #227917. I'd love to also add the Apache connector, but I shouldn't take on the responsibility of maintaining something I don't use. If you do decide to do that, please create a PR for it and add me to CC as I have thought about (and discussed with koobs@) how to do that. Build and nginx links to it without any issues. Created attachment 199378 [details]
rename of modsecurity3 (patch-compatible)
Fixed category order, so "security" is listed first. (poudriere testport)
(In reply to Marie Helene Kvello-Aune from comment #7) Given this fix poudriere testport is satisfied (ignoring unstripped libraries which I consider out of scope for this change and should be fixed with a few other things when the port is updated). (In reply to Marie Helene Kvello-Aune from comment #5) I am a part of @Apache, too. I will discuss it with him, but do not think much of moving the port to security. (In reply to Jochen Neumeister from comment #9) I'm not hung up on it being in category security. I'm more interested in the rename to distinguish it from the Apache modules and align with the product's name. Would welcome the additional "connector" ports for both Apache and Nginx! (In reply to Bernard Spil from comment #10) I'd like to see a www/modsecurity3-connector port which uses flavours to support apache and nginx. But that's not related to *this* PR, and should be dealt with in a separate PR. I had to make a few additional changes, cf https://reviews.freebsd.org/D18353: - Missing USES=gnome (pre-existing bug) - Missing CONFLICTS (partly pre-existing) - Moved MOVED entry to the bottom I just noticed that I mistyped the date in MOVED. I will correct it before I commit. (In reply to Dag-Erling Smørgrav from comment #12) Why you dump PORTREVISION on nginx-devel, not nginx? (In reply to Jochen Neumeister from comment #13) > (In reply to Dag-Erling Smørgrav from comment #12) > > > Why you dump PORTREVISION on nginx-devel, not nginx? Looks like that part of the patch fell out. Phabricator has been broken for a week or two and I was forced to create the differential manually. A commit references this bug: Author: des Date: Tue Nov 27 14:28:51 UTC 2018 New revision: 486012 URL: https://svnweb.freebsd.org/changeset/ports/486012 Log: Rename www/mod_security3 to security/modsecurity3. It is not an Apache module, and the upstream name is ModSecurity in a single word. PR: 233351 Submitted by: Marie Helene Kvello-Aune, Marius Halden (maintainer) Differential Revision: https://reviews.freebsd.org/D18353 Changes: head/MOVED head/security/Makefile head/security/modsecurity3/ head/security/modsecurity3/Makefile head/www/Makefile head/www/mod_security3/ head/www/nginx/Makefile head/www/nginx/Makefile.extmod head/www/nginx/Makefile.options.desc head/www/nginx-devel/Makefile head/www/nginx-devel/Makefile.extmod head/www/nginx-devel/Makefile.options.desc |