elharo opened a new issue, #1618:
URL: https://github.com/apache/maven-dependency-plugin/issues/1618

   ## Problem
   
   The `logUnpack()` method in `UnpackUtil.java:159-161` has inverted logic:
   
   ```java
   private void logUnpack(Log logger, File file, File location, String 
includes, String excludes) {
       if (logger.isInfoEnabled()) {
           return;  // BUG: returns when logging IS enabled
       }
       // ... builds message ...
       logger.info(msg.toString());  // never reached when logging is enabled
   }
   ```
   
   The method returns early when `isInfoEnabled()` is `true`, so log messages 
are never actually logged.
   
   ## Fix
   
   Change the condition to negate the check:
   
   ```java
   if (!logger.isInfoEnabled()) {
       return;
   }
   ```
   
   ## File
   
   
`src/main/java/org/apache/maven/plugins/dependency/utils/UnpackUtil.java:158-186`


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

Reply via email to