cstamas commented on code in PR #1982:
URL: https://github.com/apache/maven/pull/1982#discussion_r1891626090


##########
impl/maven-cli/src/main/java/org/apache/maven/cling/invoker/LookupInvoker.java:
##########
@@ -553,13 +579,19 @@ protected void settings(C context, boolean 
emitSettingsWarnings, SettingsBuilder
                 .build();
 
         customizeSettingsRequest(context, settingsRequest);
+        if (context.eventSpyDispatcher != null) {

Review Comment:
   This method is called twice, once (very early) when loading up extensions, 
then spy is null. The 2nd invocation is when "maven boots" and then spy should 
not be null...



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