Re: hidden visibility on mac (Re: [PUSHED][3-5] Re: [REVIEW 3-5] fdo#37044 crash on mac with transliteration)

2012-05-14 Thread Stephan Bergmann
On 05/10/2012 06:18 PM, Tor Lillqvist wrote: This patch makes visibility detection work for me in my 10.4 tree. The build is still going so I can't say if the result will work;) Tried a 10.4 build over the weekend, and it worked fine again for me. Thanks, Stephan

hidden visibility on mac (Re: [PUSHED][3-5] Re: [REVIEW 3-5] fdo#37044 crash on mac with transliteration)

2012-05-10 Thread Michael Stahl
On 10/05/12 12:43, Tor Lillqvist wrote: that sw can call it? don't we use default hidden visibility on Mac platform? Don't think so. At least in that Mac build tree I have that uses Xcode 3 and its gcc 4.0.1 and the 10.4 SDK, config_host.mk ends up with HAVE_GCC_VISIBILITY_FEATURE blank.

Re: hidden visibility on mac (Re: [PUSHED][3-5] Re: [REVIEW 3-5] fdo#37044 crash on mac with transliteration)

2012-05-10 Thread Tor Lillqvist
Interestingly enough, the reason why in my 10.4 Mac build tree (using Xcode 3, and thus gcc 4.0.1 and the 10.4 SDK), HAVE_VISIBILITY_FEATURE gets unset is that the checking if STL headers are visibility safe test fails. The earlier checking whether ccache /Xcode3/usr/bin/gcc-4.0

Re: hidden visibility on mac (Re: [PUSHED][3-5] Re: [REVIEW 3-5] fdo#37044 crash on mac with transliteration)

2012-05-10 Thread Stephan Bergmann
On 05/10/2012 01:06 PM, Tor Lillqvist wrote: i'm not all that familiar with how MachO linker works (other than its fancy install_names :). You mean the weird @__ stuff? I think that is something we can get rid of once we stop supporting 10.4.

Re: hidden visibility on mac (Re: [PUSHED][3-5] Re: [REVIEW 3-5] fdo#37044 crash on mac with transliteration)

2012-05-10 Thread Stephan Bergmann
On 05/10/2012 01:56 PM, Tor Lillqvist wrote: Interestingly enough, the reason why in my 10.4 Mac build tree (using Xcode 3, and thus gcc 4.0.1 and the 10.4 SDK), HAVE_VISIBILITY_FEATURE gets unset is that the checking if STL headers are visibility safe test fails. The earlier checking whether

Re: hidden visibility on mac (Re: [PUSHED][3-5] Re: [REVIEW 3-5] fdo#37044 crash on mac with transliteration)

2012-05-10 Thread Tor Lillqvist
This patch makes visibility detection work for me in my 10.4 tree. The build is still going so I can't say if the result will work;) Anyway, a summary of the changes here: - Don't hardcode the -shared, -fpic and -Wl,-z,defs options or the .so suffix for shared libraries Use platform-specific

Re: hidden visibility on mac (Re: [PUSHED][3-5] Re: [REVIEW 3-5] fdo#37044 crash on mac with transliteration)

2012-05-10 Thread Tor Lillqvist
This patch makes visibility detection work for me in my 10.4 tree. The build is still going so I can't say if the result will work;) Yes, seems to work, so pushed. Will see if it breaks something for others, or for instance with Xcode 2 and/or PPC. --tml

Re: hidden visibility on mac (Re: [PUSHED][3-5] Re: [REVIEW 3-5] fdo#37044 crash on mac with transliteration)

2012-05-10 Thread Tor Lillqvist
Yes, seems to work, so pushed. That was a bit premature, I hadn't noticed that in fact the -fvisible=hidden stuff didn't get passed to the compiler after all thanks to the order of assignments to the gb_C*FLAGS variables in the gbuild macosx,mk file. I fixed that, and now then I get linking