* [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).