Re: pkg/39120: Installing language bindings to subversion fails

["OBATA Akio" <obache%netbsd.org@localhost>]

The following reply was made to PR pkg/39120; it has been noted by GNATS.

From: "OBATA Akio" <obache%netbsd.org@localhost>
To: gnats-bugs%netbsd.org@localhost
Cc: 
Subject: Re: pkg/39120: Installing language bindings to subversion fails
Date: Thu, 10 Jul 2008 09:33:11 +0900

 >>Fix:
 > The reason is that devel/subversion-base/buildlink3.mk includes 
 > ../../security/cyrus-sasl/buildlink3.mk unconditionally. Putting it between 
 > .if !empty(PKG_OPTIONS:Msasl) fixes the problem.
 
 History of subversion-base sasl support:
 
 2008/06/19 22:33:25    1.5.0 (with sasl unconditionally)
 2008/07/03 20:39:44    1.5.0 (add sasl option, default on)
 2008/07/03 22:01:48    1.5.0nb1 (switch sasl option to default off)
 
 The 2nd situation is short term, plus with sasl option disabled is rare case.
 How about following patch?
 
 Index: buildlink3.mk
 ===================================================================
 RCS file: /cvsroot/pkgsrc/devel/subversion-base/buildlink3.mk,v
 retrieving revision 1.14
 diff -u -r1.14 buildlink3.mk
 --- buildlink3.mk      19 Jun 2008 22:33:26 -0000      1.14
 +++ buildlink3.mk      10 Jul 2008 00:26:40 -0000
 @@ -26,7 +26,16 @@
 
  .endif # SUBVERSION_BASE_BUILDLINK3_MK
 
 +_SUBVERSION_PRE_SASL_OPTION!= \
 +      if ${PKG_INFO} -qe 'subversion-base-1.5.0'; then                \
 +              ${ECHO} yes;                                            \
 +      else                                                            \
 +              ${ECHO} no;                                             \
 +      fi
 +
 +.if ${_SUBVERSION_PRE_SASL_OPTION} == "yes" || 
!empty(PKG_BUILD_OPTIONS.subversion-base:Msasl)
  .include "../../security/cyrus-sasl/buildlink3.mk"
 +.endif
 
  .if !empty(PKG_BUILD_OPTIONS.subversion-base:Mapr1)
  .  include "../../devel/apr/buildlink3.mk"
 
 
 -- 
 "Of course I love NetBSD":-)
 OBATA Akio / obache%NetBSD.org@localhost