[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r1674626 - /subversion/trunk/build/generator/gen_win_dependencies.py

From: Branko Čibej <brane_at_wandisco.com>
Date: Wed, 13 May 2015 08:24:54 +0200

On 19.04.2015 16:31, rhuijben_at_apache.org wrote:
> Author: rhuijben
> Date: Sun Apr 19 14:31:48 2015
> New Revision: 1674626
>
> URL: http://svn.apache.org/r1674626
> Log:
> On Windows: enable detecting memcache support from apr-util 1.3 or later.
>
> * build/generator/gen_win_dependencies.py
> (find_libraries): Update caller.
> (_find_apr_util_and_expat): Rename to...
> (_find_apr_util_etc): ... this. Declare availability of memcache support
> when we have apr-util 1.3 or later.
>
> Modified:
> subversion/trunk/build/generator/gen_win_dependencies.py
>
> Modified: subversion/trunk/build/generator/gen_win_dependencies.py
> URL: http://svn.apache.org/viewvc/subversion/trunk/build/generator/gen_win_dependencies.py?rev=1674626&r1=1674625&r2=1674626&view=diff
> ==============================================================================
> --- subversion/trunk/build/generator/gen_win_dependencies.py (original)
> +++ subversion/trunk/build/generator/gen_win_dependencies.py Sun Apr 19 14:31:48 2015
> @@ -296,7 +296,7 @@ class GenDependenciesBase(gen_base.Gener
>
> # Required dependencies
> self._find_apr()
> - self._find_apr_util_and_expat()
> + self._find_apr_util_etc()
> self._find_zlib()
> self._find_sqlite(show_warnings)
>
> @@ -424,7 +424,7 @@ class GenDependenciesBase(gen_base.Gener
> defines=defines,
> extra_bin=extra_bin)
>
> - def _find_apr_util_and_expat(self):
> + def _find_apr_util_etc(self):
> "Find the APR-util library and version"
>
> minimal_aprutil_version = (1, 3, 0)
> @@ -526,6 +526,13 @@ class GenDependenciesBase(gen_base.Gener
> defines=defines,
> extra_bin=extra_bin)
>
> + # Perhaps apr-util can also provide memcached support
> + if version >= (1, 3, 0) :

Just a small nit: the version check here isn't necessary; see
minimal_aprutil_version, above.

-- Brane
Received on 2015-05-13 08:25:09 CEST

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.