[ https://issues.apache.org/jira/browse/THRIFT-1824?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Jake Farrell updated THRIFT-1824: --------------------------------- Attachment: THRIFT-1824.patch configure.ac line 577 AC_CONFIG_HEADERS(config.h:config.hin) is what generates the config.h file using autoheader. This is a known problem for projects using autotools and has come up before in THRIFT-705 where we went with the addition of ifdef HAVE_CONFIG_H. There are 26 checks for this header in the lib/{cpp,c_glib,php} code. Attached is a patch which will wrap our config.h with #ifndef CONFIG_H to guard against duplicate definitions and also still allow people who would not have config.h to compile properly. > many compile warning, becase Thread.h includes config.h > ------------------------------------------------------- > > Key: THRIFT-1824 > URL: https://issues.apache.org/jira/browse/THRIFT-1824 > Project: Thrift > Issue Type: Bug > Components: C++ - Library > Affects Versions: 0.9 > Environment: 2.6.32.43 x86_64 x86_64 x86_64 GNU/Linux > automake (GNU automake) 1.9.6 > autoconf (GNU Autoconf) 2.59 > autoheader (GNU Autoconf) 2.59 > Reporter: Jian Yi > Labels: patch > Fix For: 1.2 > > Attachments: Thread.h, Thread.h.diff, THRIFT-1824.patch > > > many compile warning, becase Thread.h includes config.h > #ifdef HAVE_CONFIG_H > //#include <config.h> // fix: commented out it > #endif -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira