[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-core-devel
Subject:    Re: [PATCH] -fvisibility problems
From:       Dirk Mueller <mueller () kde ! org>
Date:       2004-11-16 22:39:03
Message-ID: 200411162339.04064.mueller () kde ! org
[Download RAW message or body]

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. 


Dirk

[prev in list] [next in list] [prev in thread] [next in thread]