Re: pkg/47388

[Thomas Klausner <wiz%NetBSD.org@localhost>]

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

From: Thomas Klausner <wiz%NetBSD.org@localhost>
To: NetBSD bugtracking <gnats-bugs%NetBSD.org@localhost>
Cc: 
Subject: Re: pkg/47388
Date: Sun, 7 Apr 2013 18:44:46 +0200

 +LDFLAGS+=               -liconv
 This looks wrong.
 Why is it needed unconditionally, and for all objects that are linked?
 In particular, it seems to work fine without PREFER_PKGSRC.
 
 I also don't understand the explanation: "The configure of cups checks
 the existence of both iconv.h and iconv_open(3) which libc.a has for
 POSIX. The configure check is wrong. In fact cups needs GNU libiconv.
 Cups tries to link libiconv_open() and the compilation failed."
 
 If the configure check is wrong, please fix that instead.
 
 Further, the diff includes changes to poppler, which are not commented
 upon at all, adding libiconv/bl3.mk. Does poppler directly need
 libiconv? If not, it's probably pulled in by some dependency and
 should be fixed in its bl3.mk instead.