git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] Link against libiconv on IRIX
@ 2010-03-29 10:57 Holger Weiß
  2010-04-05 16:45 ` Brandon Casey
  0 siblings, 1 reply; 4+ messages in thread
From: Holger Weiß @ 2010-03-29 10:57 UTC (permalink / raw)
  To: Git List

On IRIX, "-liconv" must be added to the linker command line in order to
get iconv(3) support; set the according Makefile variable appropriately.

Signed-off-by: Holger Weiß <holger@zedat.fu-berlin.de>
---
 Makefile |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/Makefile b/Makefile
index 3a6c6ea..8a0f5c4 100644
--- a/Makefile
+++ b/Makefile
@@ -919,6 +919,7 @@ ifeq ($(uname_S),IRIX)
 	SNPRINTF_RETURNS_BOGUS = YesPlease
 	SHELL_PATH = /usr/gnu/bin/bash
 	NEEDS_LIBGEN = YesPlease
+	NEEDS_LIBICONV = YesPlease
 endif
 ifeq ($(uname_S),IRIX64)
 	NO_SETENV=YesPlease
@@ -937,6 +938,7 @@ ifeq ($(uname_S),IRIX64)
 	SNPRINTF_RETURNS_BOGUS = YesPlease
 	SHELL_PATH=/usr/gnu/bin/bash
 	NEEDS_LIBGEN = YesPlease
+	NEEDS_LIBICONV = YesPlease
 endif
 ifeq ($(uname_S),HP-UX)
 	NO_IPV6=YesPlease
-- 
1.7.0.3

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] Link against libiconv on IRIX
  2010-03-29 10:57 [PATCH] Link against libiconv on IRIX Holger Weiß
@ 2010-04-05 16:45 ` Brandon Casey
  2010-04-05 17:15   ` Junio C Hamano
  2010-04-05 19:44   ` Holger Weiß
  0 siblings, 2 replies; 4+ messages in thread
From: Brandon Casey @ 2010-04-05 16:45 UTC (permalink / raw)
  To: holger; +Cc: Git List, Junio C Hamano

On 03/29/2010 05:57 AM, Holger Weiß wrote:
> On IRIX, "-liconv" must be added to the linker command line in order to
> get iconv(3) support; set the according Makefile variable appropriately.

This breaks compilation on IRIX 6.5.29m for me since there
is no separate libiconv.so.

What version of IRIX are you using?

On my system, even the iconv utility doesn't link against
a libiconv shared object.  It seems the iconv functionality is in libc.

   # ldd /usr/bin/iconv
           libc.so.1  =>    /usr/lib32/libc.so.1

Could it be that you are using a third party iconv library?
I've experienced this on another system and the problem was related
to curl.  In that case, curl was linked against an external iconv and
not the native library, so if I tried to build with curl support, I had
to also build against the external iconv library.

-brandon


> Signed-off-by: Holger Weiß <holger@zedat.fu-berlin.de>
> ---
>  Makefile |    2 ++
>  1 files changed, 2 insertions(+), 0 deletions(-)
> 
> diff --git a/Makefile b/Makefile
> index 3a6c6ea..8a0f5c4 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -919,6 +919,7 @@ ifeq ($(uname_S),IRIX)
>  	SNPRINTF_RETURNS_BOGUS = YesPlease
>  	SHELL_PATH = /usr/gnu/bin/bash
>  	NEEDS_LIBGEN = YesPlease
> +	NEEDS_LIBICONV = YesPlease
>  endif
>  ifeq ($(uname_S),IRIX64)
>  	NO_SETENV=YesPlease
> @@ -937,6 +938,7 @@ ifeq ($(uname_S),IRIX64)
>  	SNPRINTF_RETURNS_BOGUS = YesPlease
>  	SHELL_PATH=/usr/gnu/bin/bash
>  	NEEDS_LIBGEN = YesPlease
> +	NEEDS_LIBICONV = YesPlease
>  endif
>  ifeq ($(uname_S),HP-UX)
>  	NO_IPV6=YesPlease

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] Link against libiconv on IRIX
  2010-04-05 16:45 ` Brandon Casey
@ 2010-04-05 17:15   ` Junio C Hamano
  2010-04-05 19:44   ` Holger Weiß
  1 sibling, 0 replies; 4+ messages in thread
From: Junio C Hamano @ 2010-04-05 17:15 UTC (permalink / raw)
  To: Brandon Casey; +Cc: holger, Git List, Junio C Hamano

Thanks for the report.

In the meantime I'll revert 2170422 (Link against libiconv on IRIX,
2010-03-29) and wait for an improved, probably a conditional, change.

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] Link against libiconv on IRIX
  2010-04-05 16:45 ` Brandon Casey
  2010-04-05 17:15   ` Junio C Hamano
@ 2010-04-05 19:44   ` Holger Weiß
  1 sibling, 0 replies; 4+ messages in thread
From: Holger Weiß @ 2010-04-05 19:44 UTC (permalink / raw)
  To: Brandon Casey; +Cc: Git List, Junio C Hamano

* Brandon Casey <brandon.casey.ctr@nrlssc.navy.mil> [2010-04-05 11:45]:
> Could it be that you are using a third party iconv library?
> I've experienced this on another system and the problem was related
> to curl.  In that case, curl was linked against an external iconv and
> not the native library, so if I tried to build with curl support, I had
> to also build against the external iconv library.

This was exactly the problem in our case, too.  I'm sorry for the
incorrect patch.

Holger

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2010-04-05 19:45 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-03-29 10:57 [PATCH] Link against libiconv on IRIX Holger Weiß
2010-04-05 16:45 ` Brandon Casey
2010-04-05 17:15   ` Junio C Hamano
2010-04-05 19:44   ` Holger Weiß

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).