Loading STATUS +1 −11 Original line number Diff line number Diff line Loading @@ -77,17 +77,7 @@ PROPOSED PATCHES FOR THIS RELEASE: *) core: Make "Include" directives work inside previously "Include"'d files. http://people.apache.org/~colm/include_directive-1.3.patch +1: colm, wrowe -0: jim (do we *really* want those fprintfs?) Colm says: The ap_process_include_config function is a direct copy of ap_process_resource_config(), warts and all. That's where the fprintf's come from; it's existing behaviour. Jim asks: but aren't we already getting that info when we run ap_process_resource_config()? In other words, aren't we just double printing the status info? (PS: No, I haven't tried the patch yet, which, of course, would answer this directly :) ) +1: colm, wrowe, jim RELEASE NON-SHOWSTOPPERS BUT WOULD BE REAL NICE TO WRAP THESE UP: Loading Loading
STATUS +1 −11 Original line number Diff line number Diff line Loading @@ -77,17 +77,7 @@ PROPOSED PATCHES FOR THIS RELEASE: *) core: Make "Include" directives work inside previously "Include"'d files. http://people.apache.org/~colm/include_directive-1.3.patch +1: colm, wrowe -0: jim (do we *really* want those fprintfs?) Colm says: The ap_process_include_config function is a direct copy of ap_process_resource_config(), warts and all. That's where the fprintf's come from; it's existing behaviour. Jim asks: but aren't we already getting that info when we run ap_process_resource_config()? In other words, aren't we just double printing the status info? (PS: No, I haven't tried the patch yet, which, of course, would answer this directly :) ) +1: colm, wrowe, jim RELEASE NON-SHOWSTOPPERS BUT WOULD BE REAL NICE TO WRAP THESE UP: Loading