[kdelibs/frameworks] tier1/kwindowsystem/src: The config file appears in public headers, so it must be installed.

David Faure faure at kde.org
Thu May 24 22:03:40 UTC 2012


On Thursday 24 May 2012 23:57:05 Stephen Kelly wrote:
> We probably need a better solution than HAVE_X11 in the config file.

Yes. Old rule of the autoconf days: public headers MUST NOT include config.h

> Why not keep the headers not-ifdefed and ifdef the implementation
> only?

Yes, that's how it's usually done.

-- 
David Faure, faure at kde.org, http://www.davidfaure.fr
Sponsored by Nokia to work on KDE, incl. KDE Frameworks 5



More information about the Kde-frameworks-devel mailing list