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.