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

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

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
>
>  On Sat, 14 Jan 2012 02:15:05 +0900, Daniel Horecki 
> <morr%netbsd.org@localhost> wrote:
>
>  >  Module Name:        pkgsrc
>  >  Committed By:       morr
>  >  Date:               Fri Jan 13 17:13:39 UTC 2012
>  > Modified Files:
>  >      pkgsrc/devel/bison: Makefile
>  > Log Message:
>  >  Add missing converters/libiconv buildlink, fixes PR 45826.
>  > Bump PKGREVISION
>
>  I'm not sure this is the real fix.
>
>  devel/bison already buildlink with devel/gettext-lib, and it will pull 
> converters/libiconv.
>
>  If builtin gettext-lib is used, builtin libiconv must be used
>  (You can see it in in devel/gettext-lib/buildlink3.mk).
>
>  If USE_GNU_ICONV=Yes, it force to use GNU iconv from pkgsrc must be used
>  (You can see it in converters/libiconv/builtin.mk).
>
>  After your change, it result in mixed usage of builtin libiconv through 
> gettext-lib and
>  pkgsrc GNU libiconv, it is not good situation.
>
>  About this PR, USE_GNU_ICONV is not a user settable variable, so it must not 
> be set by users.
>  PREFER_PKGSRC+=libiconv should be used instead.
>
>  Anyway, this is a general issue for current {buildlink,builtin}.mk, for 
> following situation:
>   * package A depend on package B
>   * both A and B have builtin support
>   * builtin one is preferred for A and pkgsrc one is preferred for B.
>  For such situation, mixed usage must not be allowed.
>  I feel that if pkgsrc one is preferred for B, A from pkgsrc also be used.
>
>  --
>  OBATA 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= 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= 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+= 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?


-- 
Pozdrawiam, Bartosz Kuźma.