On Sat, Dec 10, 2005 at 03:33:57PM -0800, Justin Erenkrantz wrote: > Colm and I made the following change to allow a program to define an > 'acceptability' check when we're searching for APR. For httpd, this is > useful to skip APR 1.1 installs when we really just want 1.2 installs; > rather than doing that sub-version check after we found an acceptable > version. > > Anyone see any improvements we can make to this? > > Otherwise, I'll commit this and its related apr-util changes in the next > day or so.
Doing the version check using APACHE_CHECK_APxVER that early doesn't work, it breaks like PR 37283. I don't entirely undestand why but APACHE_CHECK_APxVER could be written to use `ap*-config --version` instead of CPP checks to avoid that, probably. In concept I'm not convinced this is a good idea anyway. Say I have APR version X installed in standard prefix /usr/local/apr, and a bunch of applications built which link against that. When I go to upgrade any of those applications, I want the app to either continue using that standard system APR or fail to build and tell me if I have to upgrade to APR version Y instead. I don't want each app to instead install its own bundled copy of APR and end up with N different APR installs in different prefixes. (this is particularly important when these apps may be interlinked as in httpd/Subversion via mod_dav_svn) joe
