[jira] [Commented] (THRIFT-4896) cpp and c_glib include paths are added to source files when building

["James E. King III (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/THRIFT-4896?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16870599#comment-16870599 ] 

James E. King III commented on THRIFT-4896:
-------------------------------------------

I noticed this a while back but never added anything to the backlog to track it, thank you!

> cpp and c_glib include paths are added to source files when building
> --------------------------------------------------------------------
>
>                 Key: THRIFT-4896
>                 URL: https://issues.apache.org/jira/browse/THRIFT-4896
>             Project: Thrift
>          Issue Type: Bug
>          Components: Build Process
>            Reporter: Kevin Wojniak
>            Priority: Major
>
> AM_CPPFLAGS is used by cpp and c_glib. They both add their include directories to AM_CPPFLAGS. However, since thrift header files names are generic (e.g. Processor.h) and can be reused per language, like with cpp and cocoa (as of 0.12), there is possibility for conflict due cpp and cocoa using same header file names. Automake supports target_CPPFLAGS instead which appears to limit the compile flags scope.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)