This is the way to go, so please go with the minor MMN bump. Ryan
---------------------------------------------- Ryan Bloom [EMAIL PROTECTED] 645 Howard St. [EMAIL PROTECTED] San Francisco, CA > -----Original Message----- > From: William A. Rowe, Jr. [mailto:[EMAIL PROTECTED]] > Sent: Monday, April 29, 2002 9:26 PM > To: [EMAIL PROTECTED] > Cc: [EMAIL PROTECTED]; [EMAIL PROTECTED] > Subject: Re: cvs commit: httpd-2.0/include ap_mmn.h > > T-1200 > > If I don't see any agreement that this is the way to go, I'll be happy to > revert > to the original mmn bump. > > Please comment. This should be a group decision. > > Bill > > At 07:59 AM 4/29/2002, William A. Rowe, Jr. wrote: > >Since a number of folks have complained about the major bump, I've > >reconsidered > >the impact on those testing 3rd party add-ons, and it would be good to > >avoid this > >bump due to internal changes for ssl, proxy and cgi. > > > >But the APR changes are a little more troublesome. On win32 we can live > with > >a minor bump, since the module will simply fail to load (file/entry point > >not found > >error attempting to load a module or mismatched libhttpd.dll + > libapr.dll). > > > >I'm suggesting we demote to a minor bump and ask Sander to roll this > commit > >into .36. Please comment. > > > >Bill > > > >At 07:51 AM 4/29/2002, [EMAIL PROTECTED] wrote: > >>wrowe 02/04/29 05:51:31 > >> > >> Modified: include ap_mmn.h > >> Log: > >> Demote to a minor bump for .36 - those affected by API changes > should > >> be a very small subset of modules. Here are the afflicted types & > fns; > >> > >> apr_explode_time -> apr_time_exp_tz > >> apr_explode_localtime -> apr_time_exp_lt > >> apr_get_groupname -> apr_group_name_get > >> > >> deprecated apr_lock_t, apr_os_lock_t and fns, but the warnings > >> have been there a while so no sympathy. > >> > >> added dav_hooks_search_t and DASL Search functions. > >> > >> added apr_cmdtype_e *type arg to mod_cgi opt fn ap_cgi_build_command. > >> > >> added proxy_fixups hook and ap_proxy_ssl_disable, and > >> optional fn ssl_engine_disable. > >> > >> Revision Changes Path > >> 1.43 +3 -3 httpd-2.0/include/ap_mmn.h > >> > >> Index: ap_mmn.h > >> =================================================================== > >> RCS file: /home/cvs/httpd-2.0/include/ap_mmn.h,v > >> retrieving revision 1.42 > >> retrieving revision 1.43 > >> diff -u -r1.42 -r1.43 > >> --- ap_mmn.h 13 Apr 2002 19:58:04 -0000 1.42 > >> +++ ap_mmn.h 29 Apr 2002 12:51:31 -0000 1.43 > >> @@ -103,15 +103,15 @@ > >> * 20020319 (2.0.34-dev) M_INVALID changed, plus new M_* methods for > >> RFC 3253 > >> * 20020327 (2.0.35-dev) Add parameter to quick_handler hook > >> * 20020329 (2.0.35-dev) bump for addition of freelists to bucket > API > >> - * 20020413 (2.0.36-dev) bump for new arg to opt fn > ap_cgi_build_command > >> + * 20020329.1 (2.0.36) minor bump for new arg to opt fn > >> ap_cgi_build_command > >> */ > >> > >> #define MODULE_MAGIC_COOKIE 0x41503230UL /* "AP20" */ > >> > >> #ifndef MODULE_MAGIC_NUMBER_MAJOR > >> -#define MODULE_MAGIC_NUMBER_MAJOR 20020413 > >> +#define MODULE_MAGIC_NUMBER_MAJOR 20020329 > >> #endif > >> -#define MODULE_MAGIC_NUMBER_MINOR 0 /* 0...n */ > >> +#define MODULE_MAGIC_NUMBER_MINOR 1 /* 0...n */ > >> #define MODULE_MAGIC_NUMBER MODULE_MAGIC_NUMBER_MAJOR /* > >> backward compat */ > >> > >> /** > >> > > >