GNU bug report logs - #12728
v8.20 is missing LIBICONV for factor tool

Previous Next

Package: coreutils;

Reported by: "Christian Jullien" <eligis <at> orange.fr>

Date: Thu, 25 Oct 2012 04:54:01 UTC

Severity: normal

Done: Pádraig Brady <P <at> draigBrady.com>

Bug is archived. No further changes may be made.

Full log


Message #10 received at 12728-done <at> debbugs.gnu.org (full text, mbox):

From: Pádraig Brady <P <at> draigBrady.com>
To: Christian Jullien <eligis <at> orange.fr>
Cc: 12728-done <at> debbugs.gnu.org
Subject: Re: bug#12728: v8.20 is missing LIBICONV for factor tool
Date: Thu, 25 Oct 2012 10:54:33 +0100
On 10/25/2012 05:30 AM, Christian Jullien wrote:
> Hi maintainers,
>
> Trying to compile new 8.20 version on my Solaris 10 sparc machine using gcc
> 4.7.2, I got an error on factor link:
>
> CCLD   src/factor
>
> lib/libcoreutils.a(striconv.o): In function `mem_cd_iconv':
>
> /export/home/jullien/coreutils-8.20/lib/striconv.c:55: undefined reference
> to `libiconv'

Right you are.
This is required in the new factor because of:

  proper_name_utf8 ("Torbjorn Granlund", "Torbj\303\266rn Granlund"),   \
  proper_name_utf8 ("Niels Moller", "Niels M\303\266ller")

I didn't pick up the issue on my gcc 3.4.3, solaris 10, sparc
test system because of:

  $ grep ICONV lib/config.h
  /* #undef HAVE_ICONV */
  #define HAVE_ICONV_H 1
  #define ICONV_CONST const
  #define ICONV_FLAVOR ICONV_FLAVOR_SOLARIS

To fix this up I'll add in:

diff --git a/src/local.mk b/src/local.mk
index 6a01ef1..f40f681 100644
--- a/src/local.mk
+++ b/src/local.mk
@@ -277,6 +277,7 @@ src_cat_LDADD += $(LIBICONV)
 src_cp_LDADD += $(LIBICONV)
 src_df_LDADD += $(LIBICONV)
 src_du_LDADD += $(LIBICONV)
+src_factor_LDADD += $(LIBICONV)
 src_getlimits_LDADD += $(LIBICONV)
 src_printf_LDADD += $(LIBICONV)
 src_ptx_LDADD += $(LIBICONV)

thanks!
Pádraig.




This bug report was last modified 12 years and 205 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.