libicudata in non-standard directory linking issues.

[Michael Stahl]

On 25/06/12 12:38, Jonathan Adams wrote:
> yeah, that compiled icu fine, and I didn't even have to copy the icu
> libs to the solver ...

great!

> compiled for a long time after that ...
> 
> tools_test fails though ...
> 
> If you can look through the "debug" file (sorry 1.7Mb
> https://docs.google.com/open?id=0B6o_jmGQm0dWd3VMWEc5alhJaWc ) and

[ hmm for some reason i can't look at that in my browser profile which
has my google account cookies, but i can look at it in another browser
profile that doesn't allow any google cookies... go figure... ]

> give me pointers I'd be grateful ... if you can't then I'll leave it
> as is for now and try when I get access to a new compiler/updated
> LibreOffice source.

anyway it looks like you're actually failing in sw_swdoc_test, which
means that you must have compiled Writer, which means you've come pretty
damn far :)

don't have an immediate idea why it crashes (it says in
SwDoc::GetVbaEventProcessor), but if this is just a problem in one or a
few tests, then i suggest to just comment out these tests for now, or
put them in a "ifneq ($(OS),SOLARIS)" check (edit e.g. sw/Module_sw.mk);
if on the other hand all unit tests crash somehow then you've got a
bigger problem...

to better diagnose the crashes it would help to build with debug, i.e.
the --enable-debug configure option; be aware that this will need >25GB
of space, and linking will need some GBs of RAM (you will run out of RAM
if you have less than 2GB, i think it's best to have at least 4GB but
you may get by with 2 or even less if you disable parallel build).

> On 22 June 2012 09:57, Jonathan Adams <t12nslookup at gmail.com> wrote:
>>> i'm not sure if adding a defined(__IBMC__) etc. is the right approach
>>> but if it doesn't break anything...
>>
>> they added that line for z/OS ... which, I may be wrong, I believe to
>> be an IBM system ... looking around wikipedia; I found references to
>> what defines are set when compiling on z/OS ...

hmmm z/OS is apparently why that was initially added, but you never know
what later happened to develop a dependency on this setting...

well it seems our tinderboxes can build it on Linux and Mac OS X, so
it's probably good enough, and we don't care at all if e.g. HP/UX fails
with the patch :)