You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@thrift.apache.org by "Teemu Koponen (JIRA)" <ji...@apache.org> on 2010/02/16 05:14:27 UTC
[jira] Updated: (THRIFT-705) Installed C++ header files depend on
"config.h" which may have conflicting macros with the (autoconf based) C++
project using Thrift.
[ https://issues.apache.org/jira/browse/THRIFT-705?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Teemu Koponen updated THRIFT-705:
---------------------------------
Attachment: config-h-patch.diff
A patch against the latest git master.
> 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: Library (C++)
> Environment: Any Thrift compatible C++ compiler environment.
> Reporter: Teemu Koponen
> Attachments: config-h-patch.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.
-
You can reply to this email to add a comment to the issue online.