Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 353406 - icu-4.6's configure use /bin/sh to determine ICU_ECHO_{C,N,T}
Summary: icu-4.6's configure use /bin/sh to determine ICU_ECHO_{C,N,T}
Status: RESOLVED FIXED
Alias: None
Product: Gentoo/Alt
Classification: Unclassified
Component: Prefix Support (show other bugs)
Hardware: All OS X
: High normal (vote)
Assignee: Gentoo Prefix
URL:
Whiteboard:
Keywords:
: 358103 (view as bug list)
Depends on:
Blocks:
 
Reported: 2011-02-01 14:29 UTC by Naohiro Aota
Modified: 2011-04-10 13:12 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
fix configure to use CONFIG_SHELL (icu-4.6-configure.patch,542 bytes, patch)
2011-02-08 00:01 UTC, Naohiro Aota
Details | Diff
ebuild patch to use the above patch (icu-4.6.ebuild.patch,610 bytes, patch)
2011-02-08 00:03 UTC, Naohiro Aota
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Naohiro Aota gentoo-dev 2011-02-01 14:29:21 UTC
the ebuild set "CONFIG_SHELL=${EPREFIX}/bin/sh" to "make sure we configure with the same shell as we run icu-config with, or ECHO_N, ECHO_T and ECHO_C will be wrongly defined"

However icu-4.6 configure use custom echo test (using /bin/sh), so that the ECHO_{N,T,C} is wrongly defined.


Reproducible: Always
Comment 1 Naohiro Aota gentoo-dev 2011-02-08 00:01:29 UTC
Created attachment 261763 [details, diff]
fix configure to use CONFIG_SHELL

icu-config output without the patch
% icu-config --ldflags-searchpath --ldflags-system          
-L/Users/naota/Gentoo/usr/lib \c
-lpthread -lm   \c

% (next prompt)

icu-config output with the patch
% icu-config --ldflags-searchpath --ldflags-system                                                                                                                 
-L/Users/naota/Gentoo/usr/lib -lpthread -lm
% (next prompt)
Comment 2 Naohiro Aota gentoo-dev 2011-02-08 00:03:50 UTC
Created attachment 261765 [details, diff]
ebuild patch to use the above patch
Comment 3 Fabian Groffen gentoo-dev 2011-02-08 19:52:57 UTC
Unfortunately they indeed worked around it in a broken way.  I took your patch, applied, thanks.
Comment 4 Fabian Groffen gentoo-dev 2011-04-10 13:12:47 UTC
*** Bug 358103 has been marked as a duplicate of this bug. ***