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

Dhwani Katagade commented on KAFKA-4617:
----------------------------------------

[~ecomar] thanks for your comments.
I tried to remove the following explicit dependency from build.gradle but it is 
required by the scala plugin for building test classes. Without it the 
:core:compileTestScala task fails.

testCompile project(':clients').sourceSets.test.output

I had to fix the generated .classpath file in the whenMerged closure itself. I 
have pushed an additional commit with changes to handle this.
Please review.

> gradle-generated core eclipse project has incorrect source folder structure
> ---------------------------------------------------------------------------
>
>                 Key: KAFKA-4617
>                 URL: https://issues.apache.org/jira/browse/KAFKA-4617
>             Project: Kafka
>          Issue Type: Bug
>          Components: build
>            Reporter: Edoardo Comar
>            Assignee: Dhwani Katagade
>            Priority: Minor
>              Labels: build
>
> The gradle-generated Eclipse Scala project for Kafka core has a 
> classpath defined as :
> {code:xml}
>       <classpathentry kind="src" path="src/main/scala"/>
>       <classpathentry kind="src" path="src/test/scala"/>
>       <classpathentry kind="src" path="src/test/resources"/>
> {code}
> because of how the source files are for tests are structured, code navigation 
> / running unit tests fails. The correct structure should be instead :
> {code:xml}
>       <classpathentry kind="src" path="src/main/scala"/>
>       <classpathentry excluding="integration/|other/|unit/" kind="src" 
> path="src/test/scala"/>
>       <classpathentry kind="src" path="src/test/scala/other"/>
>       <classpathentry kind="src" path="src/test/scala/integration"/>
>       <classpathentry kind="src" path="src/test/scala/unit"/>
>       <classpathentry kind="src" path="src/test/resources"/>
> {code}
> Moreover, the classpath included as libraries core/build/test and 
> core/build/resources
> which should not be there as the eclipse classes are not generated under build



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to