packages: apache/apache-bug-40970.patch, apache/apache.spec, apache/apache....

arekm arekm at pld-linux.org
Sat Sep 25 12:27:58 CEST 2010


Author: arekm                        Date: Sat Sep 25 10:27:58 2010 GMT
Module: packages                      Tag: HEAD
---- Log message:
- use upstream patch that adds -T for not doing some checks instead of dropping is_dir check in the code

---- Files affected:
packages/apache:
   apache-bug-40970.patch (1.4 -> 1.5) , apache.spec (1.603 -> 1.604) , apache.sysconfig (1.14 -> 1.15) 

---- Diffs:

================================================================
Index: packages/apache/apache-bug-40970.patch
diff -u packages/apache/apache-bug-40970.patch:1.4 packages/apache/apache-bug-40970.patch:1.5
--- packages/apache/apache-bug-40970.patch:1.4	Tue Jul 28 18:23:59 2009
+++ packages/apache/apache-bug-40970.patch	Sat Sep 25 12:27:52 2010
@@ -1,12 +1,87 @@
---- apache/server/core.c~	2008-06-06 11:05:04.956063837 +0200
-+++ apache/server/core.c	2008-06-06 11:11:35.201986198 +0200
-@@ -1176,8 +1176,7 @@
+Index: server/core.c
+===================================================================
+--- server/core.c	(wersja 984170)
++++ server/core.c	(wersja 984171)
+@@ -1183,13 +1183,19 @@
+         return err;
+     }
  
-     /* TODO: ap_configtestonly && ap_docrootcheck && */
++    /* When ap_document_root_check is false; skip all the stuff below */
++    if (!ap_document_root_check) {
++       conf->ap_document_root = arg;
++       return NULL;
++    }
++
+     /* Make it absolute, relative to ServerRoot */
+     arg = ap_server_root_relative(cmd->pool, arg);
+     if (arg == NULL) {
+         return "DocumentRoot must be a directory";
+     }
+ 
+-    /* TODO: ap_configtestonly && ap_docrootcheck && */
++    /* TODO: ap_configtestonly */
      if (apr_filepath_merge((char**)&conf->ap_document_root, NULL, arg,
--                           APR_FILEPATH_TRUENAME, cmd->pool) != APR_SUCCESS
--        || !ap_is_directory(cmd->pool, arg)) {
-+                           APR_FILEPATH_TRUENAME, cmd->pool) != APR_SUCCESS) {
-         if (cmd->server->is_virtual) {
-             ap_log_perror(APLOG_MARK, APLOG_STARTUP, 0,
-                           cmd->pool,
+                            APR_FILEPATH_TRUENAME, cmd->pool) != APR_SUCCESS
+         || !ap_is_directory(cmd->pool, arg)) {
+Index: server/main.c
+===================================================================
+--- server/main.c	(wersja 984170)
++++ server/main.c	(wersja 984171)
+@@ -361,7 +361,7 @@
+ #endif /* AP_MPM_WANT_SET_GRACEFUL_SHUTDOWN */
+ #endif
+     ap_log_error(APLOG_MARK, APLOG_STARTUP, 0, NULL,
+-                 "       %s [-v] [-V] [-h] [-l] [-L] [-t] [-S]", pad);
++                 "       %s [-v] [-V] [-h] [-l] [-L] [-t] [-T] [-S]", pad);
+     ap_log_error(APLOG_MARK, APLOG_STARTUP, 0, NULL,
+                  "Options:");
+ 
+@@ -440,10 +440,15 @@
+                  "  -M                 : a synonym for -t -D DUMP_MODULES");
+     ap_log_error(APLOG_MARK, APLOG_STARTUP, 0, NULL,
+                  "  -t                 : run syntax check for config files");
++    ap_log_error(APLOG_MARK, APLOG_STARTUP, 0, NULL,
++                "  -T                 : start without DocumentRoot(s) check");
+ 
+     destroy_and_exit_process(process, 1);
+ }
+ 
++/* Set ap_document_root_check to default value: true */
++AP_DECLARE_DATA int ap_document_root_check = 1;
++
+ int main(int argc, const char * const argv[])
+ {
+     char c;
+@@ -592,6 +597,10 @@
+             configtestonly = 1;
+             break;
+ 
++       case 'T':
++           ap_document_root_check = 0;
++           break;
++
+         case 'S':
+             configtestonly = 1;
+             new = (char **)apr_array_push(ap_server_config_defines);
+Index: include/http_main.h
+===================================================================
+--- include/http_main.h	(wersja 984170)
++++ include/http_main.h	(wersja 984171)
+@@ -32,7 +32,7 @@
+  * in apr_getopt() format.  Use this for default'ing args that the MPM
+  * can safely ignore and pass on from its rewrite_args() handler.
+  */
+-#define AP_SERVER_BASEARGS "C:c:D:d:E:e:f:vVlLtSMh?X"
++#define AP_SERVER_BASEARGS "C:c:D:d:E:e:f:vVlLtTSMh?X"
+ 
+ #ifdef __cplusplus
+ extern "C" {
+@@ -53,6 +53,8 @@
+ /** An array of all -D defines on the command line.  This allows people to
+  *  effect the server based on command line options */
+ AP_DECLARE_DATA extern apr_array_header_t *ap_server_config_defines;
++/** Available integer for using the -T switch */
++AP_DECLARE_DATA extern int ap_document_root_check;
+ 
+ /**
+  * An optional function to send signal to server on presence of '-k'

================================================================
Index: packages/apache/apache.spec
diff -u packages/apache/apache.spec:1.603 packages/apache/apache.spec:1.604
--- packages/apache/apache.spec:1.603	Wed Sep  1 12:53:46 2010
+++ packages/apache/apache.spec	Sat Sep 25 12:27:52 2010
@@ -1764,7 +1764,7 @@
 %patch8 -p1
 %patch9 -p1
 %patch10 -p1
-%patch11 -p1
+%patch11 -p0
 %patch12 -p1
 %patch14 -p1
 %patch15 -p1
@@ -2764,6 +2764,9 @@
 All persons listed below can be reached at <cvs_login>@pld-linux.org
 
 $Log$
+Revision 1.604  2010/09/25 10:27:52  arekm
+- use upstream patch that adds -T for not doing some checks instead of dropping is_dir check in the code
+
 Revision 1.603  2010/09/01 10:53:46  glen
 - revert wrong branch commit (update to 2.3.6, now tagged as DEVEL)
 

================================================================
Index: packages/apache/apache.sysconfig
diff -u packages/apache/apache.sysconfig:1.14 packages/apache/apache.sysconfig:1.15
--- packages/apache/apache.sysconfig:1.14	Fri Jan  5 17:16:55 2007
+++ packages/apache/apache.sysconfig	Sat Sep 25 12:27:52 2010
@@ -8,6 +8,7 @@
 #HTTPD_CONF="/etc/httpd/apache.conf"
 
 # other command-line options
+# -T avoid doing many unneded DocumentRoot checks
 HTTPD_OPTS=""
 
 # choose MPM
================================================================

---- CVS-web:
    http://cvs.pld-linux.org/cgi-bin/cvsweb.cgi/packages/apache/apache-bug-40970.patch?r1=1.4&r2=1.5&f=u
    http://cvs.pld-linux.org/cgi-bin/cvsweb.cgi/packages/apache/apache.spec?r1=1.603&r2=1.604&f=u
    http://cvs.pld-linux.org/cgi-bin/cvsweb.cgi/packages/apache/apache.sysconfig?r1=1.14&r2=1.15&f=u



More information about the pld-cvs-commit mailing list