[ https://issues.apache.org/jira/browse/THRIFT-1824?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13545688#comment-13545688 ]
Jian Yi commented on THRIFT-1824: --------------------------------- make[2]: Entering directory `@_@' if g++ -DHAVE_CONFIG_H -I. -I. -I../.. -I../.. -I../../../third-party/boost/include -I../../../third-party/thrift/include -I../../../third-party/sqlite/include -I../../../third-party/gflags/include -I../../../third-party/glog/include -I../../../third-party/libevent/include -I../../../third-party/mysql/include -D_GNU_SOURCE -rdynamic -Wall -fPIC -g -MT ExecutorService_server.skeleton.o -MD -MP -MF ".deps/ExecutorService_server.skeleton.Tpo" -c -o ExecutorService_server.skeleton.o `test -f './ExecutorService_server.skeleton.cpp' || echo './'`./ExecutorService_server.skeleton.cpp; \ then mv -f ".deps/ExecutorService_server.skeleton.Tpo" ".deps/ExecutorService_server.skeleton.Po"; else rm -f ".deps/ExecutorService_server.skeleton.Tpo"; exit 1; fi In file included from ../../../third-party/thrift/include/thrift/concurrency/Thread.h:28, from ../../../third-party/thrift/include/thrift/server/TServer.h:26, from ../../../third-party/thrift/include/thrift/server/TSimpleServer.h:23, from ./ExecutorService_server.skeleton.cpp:6: ../../config.h:44:1: warning: "PACKAGE" redefined In file included from ../../../third-party/thrift/include/thrift/TLogging.h:24, from ../../../third-party/thrift/include/thrift/Thrift.h:51, from ../../../third-party/thrift/include/thrift/transport/TTransport.h:23, from ../../../third-party/thrift/include/thrift/protocol/TProtocol.h:23, from ../../../third-party/thrift/include/thrift/TProcessor.h:24, from ../../../third-party/thrift/include/thrift/TDispatchProcessor.h:22, from ./ExecutorService.h:10, from ./ExecutorService_server.skeleton.cpp:4: ../../../third-party/thrift/include/thrift/config.h:263:1: warning: this is the location of the previous definition In file included from ../../../third-party/thrift/include/thrift/concurrency/Thread.h:28, from ../../../third-party/thrift/include/thrift/server/TServer.h:26, from ../../../third-party/thrift/include/thrift/server/TSimpleServer.h:23, from ./ExecutorService_server.skeleton.cpp:6: ../../config.h:47:1: warning: "PACKAGE_BUGREPORT" redefined In file included from ../../../third-party/thrift/include/thrift/TLogging.h:24, from ../../../third-party/thrift/include/thrift/Thrift.h:51, from ../../../third-party/thrift/include/thrift/transport/TTransport.h:23, from ../../../third-party/thrift/include/thrift/protocol/TProtocol.h:23, from ../../../third-party/thrift/include/thrift/TProcessor.h:24, from ../../../third-party/thrift/include/thrift/TDispatchProcessor.h:22, from ./ExecutorService.h:10, from ./ExecutorService_server.skeleton.cpp:4: ../../../third-party/thrift/include/thrift/config.h:266:1: warning: this is the location of the previous definition In file included from ../../../third-party/thrift/include/thrift/concurrency/Thread.h:28, from ../../../third-party/thrift/include/thrift/server/TServer.h:26, from ../../../third-party/thrift/include/thrift/server/TSimpleServer.h:23, from ./ExecutorService_server.skeleton.cpp:6: ../../config.h:50:1: warning: "PACKAGE_NAME" redefined In file included from ../../../third-party/thrift/include/thrift/TLogging.h:24, from ../../../third-party/thrift/include/thrift/Thrift.h:51, from ../../../third-party/thrift/include/thrift/transport/TTransport.h:23, from ../../../third-party/thrift/include/thrift/protocol/TProtocol.h:23, from ../../../third-party/thrift/include/thrift/TProcessor.h:24, from ../../../third-party/thrift/include/thrift/TDispatchProcessor.h:22, from ./ExecutorService.h:10, from ./ExecutorService_server.skeleton.cpp:4: ../../../third-party/thrift/include/thrift/config.h:269:1: warning: this is the location of the previous definition In file included from ../../../third-party/thrift/include/thrift/concurrency/Thread.h:28, from ../../../third-party/thrift/include/thrift/server/TServer.h:26, from ../../../third-party/thrift/include/thrift/server/TSimpleServer.h:23, from ./ExecutorService_server.skeleton.cpp:6: ../../config.h:53:1: warning: "PACKAGE_STRING" redefined In file included from ../../../third-party/thrift/include/thrift/TLogging.h:24, from ../../../third-party/thrift/include/thrift/Thrift.h:51, from ../../../third-party/thrift/include/thrift/transport/TTransport.h:23, from ../../../third-party/thrift/include/thrift/protocol/TProtocol.h:23, from ../../../third-party/thrift/include/thrift/TProcessor.h:24, from ../../../third-party/thrift/include/thrift/TDispatchProcessor.h:22, from ./ExecutorService.h:10, from ./ExecutorService_server.skeleton.cpp:4: ../../../third-party/thrift/include/thrift/config.h:272:1: warning: this is the location of the previous definition In file included from ../../../third-party/thrift/include/thrift/concurrency/Thread.h:28, from ../../../third-party/thrift/include/thrift/server/TServer.h:26, from ../../../third-party/thrift/include/thrift/server/TSimpleServer.h:23, from ./ExecutorService_server.skeleton.cpp:6: ../../config.h:56:1: warning: "PACKAGE_TARNAME" redefined In file included from ../../../third-party/thrift/include/thrift/TLogging.h:24, from ../../../third-party/thrift/include/thrift/Thrift.h:51, from ../../../third-party/thrift/include/thrift/transport/TTransport.h:23, from ../../../third-party/thrift/include/thrift/protocol/TProtocol.h:23, from ../../../third-party/thrift/include/thrift/TProcessor.h:24, from ../../../third-party/thrift/include/thrift/TDispatchProcessor.h:22, from ./ExecutorService.h:10, from ./ExecutorService_server.skeleton.cpp:4: ../../../third-party/thrift/include/thrift/config.h:275:1: warning: this is the location of the previous definition In file included from ../../../third-party/thrift/include/thrift/concurrency/Thread.h:28, from ../../../third-party/thrift/include/thrift/server/TServer.h:26, from ../../../third-party/thrift/include/thrift/server/TSimpleServer.h:23, from ./ExecutorService_server.skeleton.cpp:6: ../../config.h:59:1: warning: "PACKAGE_VERSION" redefined In file included from ../../../third-party/thrift/include/thrift/TLogging.h:24, from ../../../third-party/thrift/include/thrift/Thrift.h:51, from ../../../third-party/thrift/include/thrift/transport/TTransport.h:23, from ../../../third-party/thrift/include/thrift/protocol/TProtocol.h:23, from ../../../third-party/thrift/include/thrift/TProcessor.h:24, from ../../../third-party/thrift/include/thrift/TDispatchProcessor.h:22, from ./ExecutorService.h:10, from ./ExecutorService_server.skeleton.cpp:4: ../../../third-party/thrift/include/thrift/config.h:281:1: warning: this is the location of the previous definition In file included from ../../../third-party/thrift/include/thrift/concurrency/Thread.h:28, from ../../../third-party/thrift/include/thrift/server/TServer.h:26, from ../../../third-party/thrift/include/thrift/server/TSimpleServer.h:23, from ./ExecutorService_server.skeleton.cpp:6: ../../config.h:65:1: warning: "VERSION" redefined In file included from ../../../third-party/thrift/include/thrift/TLogging.h:24, from ../../../third-party/thrift/include/thrift/Thrift.h:51, from ../../../third-party/thrift/include/thrift/transport/TTransport.h:23, from ../../../third-party/thrift/include/thrift/protocol/TProtocol.h:23, from ../../../third-party/thrift/include/thrift/TProcessor.h:24, from ../../../third-party/thrift/include/thrift/TDispatchProcessor.h:22, from ./ExecutorService.h:10, from ./ExecutorService_server.skeleton.cpp:4: ../../../third-party/thrift/include/thrift/config.h:327:1: warning: this is the location of the previous definition > 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 > > > 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