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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to