[
https://issues.apache.org/jira/browse/THRIFT-705?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12905854#action_12905854
]
Ivan Murashko edited comment on THRIFT-705 at 9/3/10 7:23 AM:
--------------------------------------------------------------
The patch (config_h_patch-r991496.diff) removes all reverences about config.h
file from headers and put them into beginning of each cpp file.
was (Author: ivanm):
The patch removes all reverences about config.h file from headers and put
them into beginning of each cpp file.
> 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-r991496.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.
