[
https://issues.apache.org/jira/browse/THRIFT-705?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Roger Meier resolved THRIFT-705.
--------------------------------
Resolution: Fixed
we use the following ifdef on every place.
{noformat}
#ifdef HAVE_CONFIG_H
#include <config.h>
#endif
{noformat}
please create a new issue if there is more...
-roger
> Installed C++ header files depend on "config.h" which may have conflicting
> macros with the (autoconf based) C++ project using Thrift.
> -------------------------------------------------------------------------------------------------------------------------------------
>
> Key: THRIFT-705
> URL: https://issues.apache.org/jira/browse/THRIFT-705
> Project: Thrift
> Issue Type: Bug
> Components: C++ - Library
> Environment: Any Thrift compatible C++ compiler environment.
> Reporter: Teemu Koponen
> Attachments: config-h-patch.diff, config_h_patch-r994950.diff
>
>
> The public, installed Thrift C++ header files depend on an autoconf generated
> system specific 'config.h' header file. Unfortunately, having this header
> file as a part of the library headers is generally troublesome. For example,
> see:
> http://sources.redhat.com/autobook/autobook/autobook_96.html
> Attached is a patch that relies on an alternative, simpler approach based on
> AX_PREFIX_CONFIG_H, which adds a package prefix to both config.h file name
> and to the macros defined within:
> http://www.nongnu.org/autoconf-archive/ax_prefix_config_h.html
> The resulting thrift-config.h can be installed and included safely from C++
> applications having their own config.h.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
