[pkgsrc/trunk]: pkgsrc/devel/subversion-base Restore support for apr0 via the...

[gdt <gdt%pkgsrc.org@localhost>]

details:   https://anonhg.NetBSD.org/pkgsrc/rev/82a746bfcfd2
branches:  trunk
changeset: 394009:82a746bfcfd2
user:      gdt <gdt%pkgsrc.org@localhost>
date:      Tue Jun 02 15:34:44 2009 +0000

description:
Restore support for apr0 via the SUGGESTED apr1 option.

Do not include logic to auto-enable apr1 if the option apache22 is
defined because apache22 does not appear to be a valid option.

Do not force apr1 if serf is selected.  Instead, fail and force the
user to choose consistent options.

No PKGREVISION because apr1 is the default and the resulting package
is unchanged.  apr0 support not tested.

diffstat:

 devel/subversion-base/Makefile      |  19 ++++++++++++++++---
 devel/subversion-base/buildlink3.mk |  13 ++++++++++---
 devel/subversion-base/options.mk    |  11 +++++------
 3 files changed, 31 insertions(+), 12 deletions(-)

diffs (82 lines):

diff -r 633f661bd529 -r 82a746bfcfd2 devel/subversion-base/Makefile
--- a/devel/subversion-base/Makefile    Tue Jun 02 14:51:03 2009 +0000
+++ b/devel/subversion-base/Makefile    Tue Jun 02 15:34:44 2009 +0000
@@ -1,4 +1,4 @@
-# $NetBSD: Makefile,v 1.64 2009/06/01 18:56:08 gdt Exp $
+# $NetBSD: Makefile,v 1.65 2009/06/02 15:34:44 gdt Exp $
 
 PKGNAME=       subversion-base-${SVNVER}
 COMMENT=       Version control system, base programs and libraries
@@ -17,8 +17,21 @@
 CONFIGURE_ARGS+=       --disable-neon-version-check
 CONFIGURE_ARGS+=       --without-apxs
 
-.include "../../devel/apr/buildlink3.mk"
-.include "../../devel/apr-util/buildlink3.mk"
+.if !empty(PKG_OPTIONS:Mapr1)
+APU_OPTIONS=           PKG_BUILD_OPTIONS.apr-util
+.else
+APU_OPTIONS=           PKG_BUILD_OPTIONS.apr
+.endif
+BUILD_DEFS+=           ${APU_OPTIONS}
+
+.if !empty(PKG_OPTIONS:Mapr1)
+.  include "../../devel/apr/buildlink3.mk"
+.  include "../../devel/apr-util/buildlink3.mk"
+.else
+BUILDLINK_API_DEPENDS.apr+=    apr>=0.9.5
+.  include "../../devel/apr0/buildlink3.mk"
+.endif
+
 BUILDLINK_API_DEPENDS.sqlite3+=        sqlite3>=3.4
 .include "../../databases/sqlite3/buildlink3.mk"
 .include "../../devel/zlib/buildlink3.mk"
diff -r 633f661bd529 -r 82a746bfcfd2 devel/subversion-base/buildlink3.mk
--- a/devel/subversion-base/buildlink3.mk       Tue Jun 02 14:51:03 2009 +0000
+++ b/devel/subversion-base/buildlink3.mk       Tue Jun 02 15:34:44 2009 +0000
@@ -1,4 +1,4 @@
-# $NetBSD: buildlink3.mk,v 1.20 2009/05/30 00:16:47 gdt Exp $
+# $NetBSD: buildlink3.mk,v 1.21 2009/06/02 15:34:44 gdt Exp $
 
 .include "../../mk/bsd.fast.prefs.mk"
 .include "../../devel/subversion/Makefile.version"
@@ -21,8 +21,15 @@
 .include "../../security/cyrus-sasl/buildlink3.mk"
 .endif
 
-.include "../../devel/apr/buildlink3.mk"
-.include "../../devel/apr-util/buildlink3.mk"
+.if !empty(PKG_BUILD_OPTIONS.subversion-base:Mapr1)
+.  include "../../devel/apr/buildlink3.mk"
+.  include "../../devel/apr-util/buildlink3.mk"
+.else
+.  include "../../devel/apr0/buildlink3.mk"
+.endif
+
+# If serf and -apr1 are selected, the build will probably fail.
+# Do that rather than force neon for apr0
 .if !empty(PKG_BUILD_OPTIONS.subversion-base:Mserf)
 .  include "../../www/serf/buildlink3.mk"
 .else
diff -r 633f661bd529 -r 82a746bfcfd2 devel/subversion-base/options.mk
--- a/devel/subversion-base/options.mk  Tue Jun 02 14:51:03 2009 +0000
+++ b/devel/subversion-base/options.mk  Tue Jun 02 15:34:44 2009 +0000
@@ -1,13 +1,12 @@
-# $NetBSD: options.mk,v 1.9 2009/05/30 00:16:47 gdt Exp $
+# $NetBSD: options.mk,v 1.10 2009/06/02 15:34:44 gdt Exp $
 
 PKG_OPTIONS_VAR=       PKG_OPTIONS.subversion
-PKG_SUPPORTED_OPTIONS= serf sasl
-PKG_SUGGESTED_OPTIONS= # empty
+PKG_SUPPORTED_OPTIONS= apr1 serf sasl
+PKG_SUGGESTED_OPTIONS= apr1
 
 # Note that this file is included as part of several packages.
-# Therefore this file only defines options, but does not include the
-# typical fragments to change behavior based on options, leaving those
-# to the individual Makefiles.
+# Therefore this file defines options and includes some but not all of
+# the typical fragments, leaving some to individual package files.
 
 .include "../../mk/bsd.options.mk"