[Buildroot] [PATCH] package: icu: detect and add compiler symbol prefix to the assembly code

[Sonic Zhang]

Hi Peter,

On Wed, Jan 15, 2014 at 5:52 AM, Peter Korsgaard <jacmet at uclibc.org> wrote:
>>>>>> "sonic" == sonic adi <sonic.adi at gmail.com> writes:
>
>  > From: Sonic Zhang <sonic.zhang at analog.com>
>  > Some compiler, such as Blackfin GNU compiler, prefix a charater to any
>  > C symbol in generated assembly code. If any assembly symbol is invoked
>  > from C code, it needs to be prefixed as well.
>
>  > Signed-off-by: Sonic Zhang <sonic.zhang at analog.com>
>
> Thanks. Has this been submitted upstream?

Do you mean the icu source upstream? Not yet. Where do you prefer to
host this patch? in buildroot or icu project?

>
>  > ---
>  >  .../icu/icu-detect-compiler-symbol-prefix.patch    |  104 ++++++++++++++++++++
>  >  1 file changed, 104 insertions(+)
>  >  create mode 100644 package/icu/icu-detect-compiler-symbol-prefix.patch
>
>  > diff --git a/package/icu/icu-detect-compiler-symbol-prefix.patch b/package/icu/icu-detect-compiler-symbol-prefix.patch
>  > new file mode 100644
>  > index 0000000..9ca7690
>  > --- /dev/null
>  > +++ b/package/icu/icu-detect-compiler-symbol-prefix.patch
>  > @@ -0,0 +1,104 @@
>  > +package: icu: detect and add compiler symbol prefix to the assembly code
>  > +
>  > +Some compiler, such as Blackfin GNU compiler, prefix a charater to any
>  > +C symbol in generated assembly code. If any assembly symbol is invoked
>  > +from C code, it needs to be prefixed as well.
>  > +
>  > +--- icu-51.2/source/configure       2014-01-06 17:13:20.907249812 +0800
>  > ++++ icu-51.2.bak/source/configure   2014-01-06 17:12:57.335248659 +0800
>
> Please adjust configure.ac and set ICU_AUTORECONF = YES instead.

But, there is no configure.ac file in the icu source code.

Regards,

Sonic
>
> --
> Bye, Peter Korsgaard