tomaswolf commented on code in PR #579:
URL: https://github.com/apache/mina-sshd/pull/579#discussion_r1711052910


##########
pom.xml:
##########
@@ -698,16 +698,6 @@
             <artifactId>logback-classic</artifactId>
             <scope>test</scope>
         </dependency>
-        <dependency>
-            <groupId>junit</groupId>
-            <artifactId>junit</artifactId>
-            <scope>test</scope>
-        </dependency>
-        <dependency>
-            <groupId>org.mockito</groupId>
-            <artifactId>mockito-core</artifactId>
-            <scope>test</scope>
-        </dependency>

Review Comment:
   Found another way that seems to work: leave all other poms unchanged, keep 
the junit dependency in the parent POM. In sshd-benchmarks, override it by 
adding an explicit dependency:
   ```
           <dependency>
               <groupId>junit</groupId>
               <artifactId>junit</artifactId>
               <scope>provided</scope>
           </dependency>
   ```
   I actually had tried that first and it hadn't worked. I'll have re-verify on 
my main development machine.



-- 
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.

To unsubscribe, e-mail: dev-unsubscr...@mina.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@mina.apache.org
For additional commands, e-mail: dev-h...@mina.apache.org

Reply via email to