[
https://issues.apache.org/jira/browse/THRIFT-3784?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15312856#comment-15312856
]
ASF GitHub Bot commented on THRIFT-3784:
----------------------------------------
Github user jeking3 commented on the issue:
https://github.com/apache/thrift/pull/984
The project is still accepting patches, however as you can see we have over
100 pull requests (I have 3 in the queue) but we haven't been able to stabilize
the apache build job for quite some time, leading to a backlog. Only a few
people have access to the apache build job and until we can get builds to pass
the project lead decided (correctly, in my opinion) not to merge any more code.
> thrift-maven-plugin generates invalid include directories for IDL in
> dependency JARs
> ------------------------------------------------------------------------------------
>
> Key: THRIFT-3784
> URL: https://issues.apache.org/jira/browse/THRIFT-3784
> Project: Thrift
> Issue Type: Bug
> Reporter: Chris Osborn
>
> The thrift-maven-plugin attempts to support building .thrift files that
> depend on other .thrift files provided in a dependency JAR. It does this by
> copying the dependency JAR's thrift files into a temporary directory then
> passing corresponding include directory options to the thrift compiler.
> However, the process of generating the include directory paths is faulty:
> there is a path for each and every directory that contains a .thrift file,
> rather than a single path to the root of the thrift file hierarchy. That
> means that any thrift files that reference others using paths relative to a
> root will fail. It may happen to work where the thrift files are all in a
> single directory, but not in the general case.
> There are no tests of this support, so I've added a few in the pull request
> I'm preparing.
> Somebody found the exact same problem and issued a pull request against this
> plugin's original repository on Github:
> https://github.com/dtrott/maven-thrift-plugin/pull/28. It went nowhere
> because the project was already pending ASF acceptance at that point, but
> it's worth looking at as an additional reference.
> Thanks!
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
