Re: PR/45826 CVS commit: pkgsrc/devel/bison

[Bartosz Kuźma <bartosz.kuzma%gmail.com@localhost>]

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

From: =?UTF-8?Q?Bartosz_Ku=C5=BAma?= <bartosz.kuzma%gmail.com@localhost>
To: gnats-bugs%netbsd.org@localhost
Cc: morr%netbsd.org@localhost, gnats-admin%netbsd.org@localhost, 
pkgsrc-bugs%netbsd.org@localhost
Subject: Re: PR/45826 CVS commit: pkgsrc/devel/bison
Date: Sat, 14 Jan 2012 12:16:45 +0100

 On Sat, Jan 14, 2012 at 06:35, OBATA Akio <obache%netbsd.org@localhost> wrote:
 > The following reply was made to PR pkg/45826; it has been noted by GNATS.
 >
 > From: "OBATA Akio" <obache%netbsd.org@localhost>
 > To: gnats-bugs%netbsd.org@localhost
 > Cc:
 > Subject: Re: PR/45826 CVS commit: pkgsrc/devel/bison
 > Date: Sat, 14 Jan 2012 14:31:52 +0900
 >
 > =C2=A0On Sat, 14 Jan 2012 02:15:05 +0900, Daniel Horecki 
 > <morr%netbsd.org@localhost=
 > wrote:
 >
 > =C2=A0> =C2=A0Module Name: =C2=A0 =C2=A0 =C2=A0 =C2=A0pkgsrc
 > =C2=A0> =C2=A0Committed By: =C2=A0 =C2=A0 =C2=A0 morr
 > =C2=A0> =C2=A0Date: =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 Fri =
 Jan 13 17:13:39 UTC 2012
 > =C2=A0> Modified Files:
 > =C2=A0> =C2=A0 =C2=A0 =C2=A0pkgsrc/devel/bison: Makefile
 > =C2=A0> Log Message:
 > =C2=A0> =C2=A0Add missing converters/libiconv buildlink, fixes PR 45826.
 > =C2=A0> Bump PKGREVISION
 >
 > =C2=A0I'm not sure this is the real fix.
 >
 > =C2=A0devel/bison already buildlink with devel/gettext-lib, and it will p=
 ull converters/libiconv.
 >
 > =C2=A0If builtin gettext-lib is used, builtin libiconv must be used
 > =C2=A0(You can see it in in devel/gettext-lib/buildlink3.mk).
 >
 > =C2=A0If USE_GNU_ICONV=3DYes, it force to use GNU iconv from pkgsrc must =
 be used
 > =C2=A0(You can see it in converters/libiconv/builtin.mk).
 >
 > =C2=A0After your change, it result in mixed usage of builtin libiconv thr=
 ough gettext-lib and
 > =C2=A0pkgsrc GNU libiconv, it is not good situation.
 >
 > =C2=A0About this PR, USE_GNU_ICONV is not a user settable variable, so it=
  must not be set by users.
 > =C2=A0PREFER_PKGSRC+=3Dlibiconv should be used instead.
 >
 > =C2=A0Anyway, this is a general issue for current {buildlink,builtin}.mk,=
  for following situation:
 > =C2=A0 * package A depend on package B
 > =C2=A0 * both A and B have builtin support
 > =C2=A0 * builtin one is preferred for A and pkgsrc one is preferred for B=
 .
 > =C2=A0For such situation, mixed usage must not be allowed.
 > =C2=A0I feel that if pkgsrc one is preferred for B, A from pkgsrc also be=
  used.
 >
 > =C2=A0--
 > =C2=A0OBATA Akio / obache%NetBSD.org@localhost
 >
 
 Use converters/libiconv from pkgsrc is a little bit odd and unclear.
 My goal is to have converters/php-iconv build against GNU libiconv (I
 need translit, wich is not available in NetBSD libiconv).
 
 I tried two possibilities:
 PREFER.iconv=3D pkgsrc - which build correctly (but I found a problem
 with chat/libpurple, which cause broken libjabber.so but with
 USE_GNU_ICONV everything works fine)
 USE_GNU_ICONV=3D Yes - it also work
 
 I tried to rebuild all packages with only USE_GNU_ICONV. I have about
 80 total packages and all of them seems to work fine. With one problem
 during build deve/bison.
 
 And then I set only PREFER_PKGSRC+=3D libiconv but it ends with
 converters/php-iconv with builtin libiconv.
 
 My question is how is the proper way to build converters/php-iconv
 with GNU libiconv?
 
 
 --=20
 Pozdrawiam, Bartosz Ku=C5=BAma.