Summary: | add www-servers/nginx support to app-admin/webapp-config | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Leonid Evdokimov <leon+gentoo> |
Component: | New packages | Assignee: | Gentoo Web Application Packages Maintainers <web-apps> |
Status: | RESOLVED FIXED | ||
Severity: | enhancement | CC: | aidanamarks, bugs_gentoo_org.korobkov, bugs_gentoo_org.Tim_OKelly, chewi, domen, earcar, email, gef.kornflakes, gentoo, hongqn, jdavid.ibp, mail, quantumsummers, sirkonst, steeeeeveee, stevan, yaleks |
Priority: | Lowest | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: |
Patch to add nginx to webapp-config
webapp-config-1.50.16-nginx.patch webapp-config-1.50.16-nginx.patch patch for nginx & gatling (httpd servers) patch for nginx & gatling (httpd servers) Revised patch for webapp-config-1.50.16-r1 |
Description
Leonid Evdokimov
2008-03-01 16:52:06 UTC
Could kill aolserver at the same time, removed. Created attachment 147826 [details, diff]
Patch to add nginx to webapp-config
I am testing nginx in a few vservers and needed support in webapp-config. Attached is a small patch to add support
Grateful if it could be pushed into an upstream release fairly promptly since it's pretty trivial
Cheers
Bump - please commit trivial patch... (In reply to comment #2) Thanks a lot, works fine Created attachment 174608 [details, diff]
webapp-config-1.50.16-nginx.patch
Here the patch in his naked form without any lib64 stuff in it. Can be applied to the source before compiling. I reordered the insert of nginx so that the resulting patch only inserts code and does not replace any existing code/line.
Can someone commit this already... It's trivial, it's annoying that it's not committed and nginx is fairly popular!! Created attachment 182771 [details, diff]
webapp-config-1.50.16-nginx.patch
Added patching of /etc/vhosts/webapp-config to include "nginx" in the list of available web servers.
Hiw about marking as FIXED this good old one ? (for testing purpose, i have just achieved a dev-db/phpmyadmin-3.1.2 installation with www-servers/nginx ; worked flawlessly ; webapp-config is imho a fantastic tool). I vote for finally closing this bug too... it works well and is IMHO a non-critical change. Regards, Elias P. Created attachment 190808 [details, diff]
patch for nginx & gatling (httpd servers)
This patch incorporates what the initial patch did plus it adds support for www-servers/gatling. www-servers/aolserver was removed since it's not longer in the tree.
Created attachment 190810 [details, diff]
patch for nginx & gatling (httpd servers)
left out 1 line on last patch that removes aolserver.
The latter patch isn't build cleanly, doesn't apply correctly for me - can't find the second (+) filenames (one by steveb is fine). I vote for applying and closing this bug already. Created attachment 192728 [details, diff]
Revised patch for webapp-config-1.50.16-r1
This patch is in response to latter comment. This patch was tested on stable webapp-config release and should apply cleanly from package's root folder.
Please implement this patch. please please please (In reply to comment #15) > please please please > Please what? If we wait another bunch of months, then this bug will be here for two years. I think the chance that we will have this bug report here so long is pretty high. That's the way it is. Unfortunately. Anyway. As a Gentoo user you should know how to make a local overlay and make a new revision of webapp-config and add the patch from here to your own ebuild. Just do that and be happy. Maybe the maintainers of nginx (Konstantin V. Arkhipov <voxus@gentoo.org> or Tobias Heinlein <keytoaster@gentoo.org>) should be addressed regarding nginx and webapp-config? They sure have more weight in Gentoo land then we as users have. Already available in my overlay: http://gitorious.org/okelly_overlay/mainline/trees/master/app-admin/webapp-config There is also other stuff from this bugzilla, I found useful for me. So welcome :) (In reply to comment #13) > Created an attachment (id=192728) [details] > Revised patch for webapp-config-1.50.16-r1 > > This patch is in response to latter comment. This patch was tested on stable > webapp-config release and should apply cleanly from package's root folder. > Thanks :) works for me So, what's the status here? (In reply to comment #19) > So, what's the status here? > Probably not much. It is lurking here in g.b.o since two years and will probably continue to do so. Do you have any issues applying the patch to your local overlay? If not then just continue doing that and wait till one of the Gentoo developers finds the time to include that trivial patch to stock webapp-config. Added patch in iElectric overlay This looks good, I agree it should be committed mainline being that nginx is available to all nowadays also... +*webapp-config-1.50.16-r4 (19 Dec 2010) + + 19 Dec 2010; Rafael G. Martins <rafaelmartins@gentoo.org> + -webapp-config-1.50.16-r3.ebuild, +webapp-config-1.50.16-r4.ebuild, + +files/webapp-config-1.50.16-update-servers.patch: + Added support to www-servers/{nginx,gatling}. Thanks to Fernando V. (bug + #211996) + |