-
Notifications
You must be signed in to change notification settings - Fork 9
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Gradle build: allow selection of a customized web.xml #32
Comments
What exactly is the proplem with PHPCGIServlet?
|
Yes it should, but from stackoverflow, looks like it doesn't... And most installation problems comes from a missing 'php-cgi'... I'll make a test soon to confirm. So let's wait a bit. |
If it doesn't do its duty as documented in the code, then its a bug and we
should try to find and fix it, rather than artificially excluding the
servlet.
2017-02-16 14:12 GMT+01:00 Sébastien Vanvelthem <notifications@github.com>:
… Yes it should, but from stackoverflow, looks like it doesn't...
And most installation problems comes from a missing 'php-cgi'... I'll make
a test soon to confirm. So let's wait a bit.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#32 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AIA126AtakNJEn_5WdYHSgi225MJKB1cks5rdEtZgaJpZM4MB-hT>
.
|
Yes I got it, but my point is a bit different:
That's why I think the possibility should be offered when building (gradle.build). This way it also help me to product clear documentation... |
Currently the gradle build includes by default the web.xml stored in the webapp folder.
By doing so the build register the PHPCGIServlet for every build (which requires a php-cgi binary installed and make the install more difficult).
As the PHPCGIServlet is not technically required for common usage, it should be possible to pass a parameter to the gradle command to select a different web.xml (without registration of PHPCGIServlet) for building the war and document it.
The text was updated successfully, but these errors were encountered: