On Mon, Nov 03, 2014 at 09:10:25AM +0100, David Coppa wrote:
> On Sun, Nov 2, 2014 at 2:34 PM, Matthieu Herrb <[email protected]> wrote:
> > On Sat, Nov 01, 2014 at 10:02:41PM +0100, David Coppa wrote:
> >> On Sat, 01 Nov 2014, Matthieu Herrb wrote:
> >>
> >> > Hi,
> >> >
> >> > icu4c 54.1p1 fails to build on arm (I'm using an imx6 but it shouldn't
> >> > matter).
> >>
> >>
> >> > gmake[2]: Leaving directory '/usr/obj/ports/icu4c-54.1/build-arm/data'
> >> > Unpacking /usr/obj/ports/icu4c-54.1/icu/source/data/in/icudt54l.dat and
> >> > generating out/tmp/icudata.lst (list of data files)
> >> > LD_LIBRARY_PATH=../lib:../stubdata:../tools/ctestfw:$LD_LIBRARY_PATH
> >> > ../bin/icupkg -d ./out/build/icudt54l --list -x \*
> >> > /usr/obj/ports/icu4c-54.1/icu/source/data/in/icudt54l.dat -o
> >> > out/tmp/icudata.lst ^^^^^^^^
> >> > usage: icupkg [-h|-?|--help ] [-tl|-tb|-te] [-c] [-C comment]
> >> > [-a list] [-r list] [-x list] [-l [-o outputListFileName]]
> >> > [-s path] [-d path] [-w] [-m mode]
> >> > [--auto_toc_prefix] [--auto_toc_prefix_with_type] [--toc_prefix]
> >> > infilename [outfilename]
> >> > echo timestamp > build-local
> >> > LD_LIBRARY_PATH=../stubdata:../tools/ctestfw:../lib:$LD_LIBRARY_PATH
> >> > ../bin/pkgdata -O ../data/icupkg.inc -q -c -s
> >> > /usr/obj/ports/icu4c-54.1/build-arm/data/out/build/icudt54l -d ./out -e
> >> > icudt54 -T ./out/tmp -p icudt54l -m common -r 8.0 ./out/tmp/icudata.lst
> >> > Error opening <./out/tmp/icudata.lst>.
> >>
> >> Antoine, I think we need to restore that chunk where it was before...
> >>
> >> Matthieu please try the following diff:
> >
> > Yes, with this patch icu4c built ok on my sabre light board.
> > Thanks.
>
> So... Ok to commit?
yes.
>
> >>
> >> Index: Makefile
> >> ===================================================================
> >> RCS file: /cvs/ports/textproc/icu4c/Makefile,v
> >> retrieving revision 1.41
> >> diff -u -p -u -p -r1.41 Makefile
> >> --- Makefile 17 Oct 2014 12:36:33 -0000 1.41
> >> +++ Makefile 1 Nov 2014 20:53:00 -0000
> >> @@ -5,12 +5,17 @@ DIST_SUBDIR= icu4c
> >>
> >> NOT_FOR_ARCHS= ${GCC3_ARCHS}
> >>
> >> +# avoid segfaults from icupkg compiled and then used during the build
> >> +.if ${MACHINE_ARCH} == "arm"
> >> +CFLAGS+= -O1 -fno-stack-protector
> >> +.endif
> >> +
> >> COMMENT= International Components for Unicode
> >>
> >> VERSION= 54.1
> >> DISTNAME= icu4c-${VERSION:S/./_/g}-src
> >> PKGNAME= icu4c-${VERSION}
> >> -REVISION= 1
> >> +REVISION= 2
> >>
> >> MAJ_V= ${VERSION:R}
> >> SO_VERSION= 8.0
> >>
--
Matthieu Herrb
