Re: [PATCH] add neon 0.28 support

2008-02-08 22:26:13 Joe Orton napisał(a):
> 0.28.0 is API/ABI-backwards-compatible with 0.27.x so no big surprises
> expected here; passes davautocheck.
>
> [[[
> Add support for neon 0.28.x:
>
> * configure.in: Add neon 0.28.0 to list of supported versions, and
> list 0.28.0 as recommended.
>
> * build/ac-macros/neon.m4 (SVN_NEON_CONFIG, SVN_LIB_NEON): Define
> SVN_NEON_0_28 for neon 0.28; extend supported version lists by one.
> ]]]
>
> Index: build/ac-macros/neon.m4
> ===================================================================
> --- build/ac-macros/neon.m4 (revision 29238)
> +++ build/ac-macros/neon.m4 (working copy)
> @@ -50,16 +50,21 @@
> NEON_VERSION=`cat $abs_srcdir/neon/.version`
> AC_MSG_RESULT([$NEON_VERSION])
>
> - if test -n ["`echo "$NEON_VERSION" | grep '^0\.2[6-8]\.'`"] ; then
> + if test -n ["`echo "$NEON_VERSION" | grep '^0\.2[6-9]\.'`"] ; then
> AC_DEFINE_UNQUOTED([SVN_NEON_0_26], [1],
> [Define to 1 if you have Neon 0.26 or later.])
> fi
>
> - if test -n ["`echo "$NEON_VERSION" | grep '^0\.2[7-8]\.'`"] ; then
> + if test -n ["`echo "$NEON_VERSION" | grep '^0\.2[7-9]\.'`"] ; then
> AC_DEFINE_UNQUOTED([SVN_NEON_0_27], [1],
> [Define to 1 if you have Neon 0.27 or later.])
> fi
>
> + if test -n ["`echo "$NEON_VERSION" | grep '^0\.2[8-9]\.'`"] ; then
> + AC_DEFINE_UNQUOTED([SVN_NEON_0_28], [1],
> + [Define to 1 if you have Neon 0.28 or later.])
> + fi
> +
> for svn_allowed_neon in $NEON_ALLOWED_LIST; do
> if test "$NEON_VERSION" = "$svn_allowed_neon" ||
> test "$svn_allowed_neon" = "any"; then
> @@ -128,16 +133,21 @@
> NEON_VERSION=`$neon_config --version | sed -e 's/^neon //'`
> AC_MSG_RESULT([$NEON_VERSION])
>
> - if test -n ["`echo "$NEON_VERSION" | grep '^0\.2[6-8]\.'`"] ; then
> + if test -n ["`echo "$NEON_VERSION" | grep '^0\.2[6-9]\.'`"] ; then
> AC_DEFINE_UNQUOTED([SVN_NEON_0_26], [1],
> [Define to 1 if you have Neon 0.26 or later.])
> fi
>
> - if test -n ["`echo "$NEON_VERSION" | grep '^0\.2[7-8]\.'`"] ; then
> + if test -n ["`echo "$NEON_VERSION" | grep '^0\.2[7-9]\.'`"] ; then
> AC_DEFINE_UNQUOTED([SVN_NEON_0_27], [1],
> [Define to 1 if you have Neon 0.27 or later.])
> fi
>
> + if test -n ["`echo "$NEON_VERSION" | grep '^0\.2[8-9]\.'`"] ; then
> + AC_DEFINE_UNQUOTED([SVN_NEON_0_28], [1],
> + [Define to 1 if you have Neon 0.28 or later.])
> + fi
> +
> for svn_allowed_neon in $NEON_ALLOWED_LIST; do
> if test "$NEON_VERSION" = "$svn_allowed_neon" ||
> test "$svn_allowed_neon" = "any"; then
> Index: configure.ac
> ===================================================================
> --- configure.ac (revision 29238)
> +++ configure.ac (working copy)
> @@ -69,8 +69,8 @@
>
> # Either a space-separated list of allowable Neon versions, or "any" to
> # mean allow anything.
> -NEON_ALLOWED_LIST="0.25.0 0.25.1 0.25.2 0.25.3 0.25.4 0.25.5 0.26.0 0.26.1 0.26.2 0.26.3 0.26.4 0.27.2"
> -NEON_RECOMMENDED_VER=0.27.2
> +NEON_ALLOWED_LIST="0.25.0 0.25.1 0.25.2 0.25.3 0.25.4 0.25.5 0.26.0 0.26.1 0.26.2 0.26.3 0.26.4 0.27.2 0.28.0"
> +NEON_RECOMMENDED_VER=0.28.0
> NEON_URL="http://www.webdav.org/neon/neon-${NEON_RECOMMENDED_VER}.tar.gz"
> dnl You can skip the neon version check only if you know what you are doing
> AC_ARG_ENABLE(neon-version-check,
>

I have been using Neon trunk for a long time and I'm confirming that Neon 0.28 works
correctly.

Also please nominate this patch for backporting to 1.5.x branch.

-- 
Arfrever Frehtes Taifersar Arahesis