[PATCH] -fvisibility problems

[Szombathelyi György]

Szombathelyi György wrote:
> Dirk Mueller wrote:
> 
>> On Tuesday 16 November 2004 21:18, Szombathelyi György wrote:
>>
>>
>>>> That cannot be committed as is, for two reasons: installed header files
>>>> must not include config.h and kjs must not have dependencies on other
>>>> parts of kdelibs.
>>>
>>>
>>> Have you got a better idea for the feature test?
>>
>>
>>
>> Well, I'm not sure yet that it is necessary, I think older gcc's can 
>> cope just fine with unsupported attributes. If not, we can either make 
>> the configure check use -fvisibility based on the gcc version or 
>> create an installable kdelibs_config.h that only contains the 
>> KDE_EXPORT macro. Not sure which variant is best.. I'm not yet 
>> convinced that the backportings of the -fvisibility stuff in the 
>> various vendor compilers is bug free.
>>
> As I have nearly finished with arts, and compiled kdelibs with 
> -fvisibility patched gcc 3.4.2, I didn't notice any problems yet.
> Some solution needed, because the current situtation breaks the compile 
> process. The -fvisibility=hidden flag added, but the KDE_EXPORT macro 
> not defined correctly.
> What about passing a -DHAVE_HIDDEN_VISIBILITY option with the 
> -fvisibility=hidden flag, this doesn't require config.h?
> 
The attaced patch does the trick, is it acceptable?
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: kdelibs-visibility2.diff
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20041117/fb3fdc5b/attachment.ksh>