ctubbsii commented on issue #1314: ZOOKEEPER-3791: Miscellaneous Maven fixes
URL: https://github.com/apache/zookeeper/pull/1314#issuecomment-611877120
 
 
   > I fail to import this pull to Eclipse. The errors with Maven still remain.
   
   Please take a look at #1313. You will need that change in addition to this 
one in order to import as an  "Existing Maven Project" in Eclipse. You will 
also need to pre-generate the sources, as described in my comment there.
   
   There remains a bug in Eclipse, where it does not properly set up the 
test-jar dependency on the class path for the zookeeper-it module. I've tried 
multiple ways of specifying the jar, without getting too crazy. It appears to 
be related to an intentional change in Eclipse 
(https://bugs.eclipse.org/bugs/show_bug.cgi?id=526858#c10). However, that jar 
is only used for a single TestUtils class, which can easily be replaced with 
commons-io (for recursively deleting temporary directories during a cleanup 
phase of a test). I intend to fix that in a subsequent pull request.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to