Skip to content

Commit 50198dd

Browse files
authored
feat: ability to avoid CLIRR dependency resolution (#959)
1 parent 6a8e98d commit 50198dd

File tree

1 file changed

+33
-15
lines changed

1 file changed

+33
-15
lines changed

java-shared-config/pom.xml

Lines changed: 33 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -345,21 +345,6 @@
345345
</execution>
346346
</executions>
347347
</plugin>
348-
<plugin>
349-
<groupId>org.codehaus.mojo</groupId>
350-
<artifactId>clirr-maven-plugin</artifactId>
351-
<configuration>
352-
<ignoredDifferencesFile>clirr-ignored-differences.xml</ignoredDifferencesFile>
353-
<logResults>true</logResults>
354-
</configuration>
355-
<executions>
356-
<execution>
357-
<goals>
358-
<goal>check</goal>
359-
</goals>
360-
</execution>
361-
</executions>
362-
</plugin>
363348
<plugin>
364349
<groupId>org.codehaus.mojo</groupId>
365350
<artifactId>animal-sniffer-maven-plugin</artifactId>
@@ -689,5 +674,38 @@
689674
</repository>
690675
</repositories>
691676
</profile>
677+
<profile>
678+
<id>clirr-compatibility-check</id>
679+
<!--
680+
CLIRR Maven plugin's dependencies are quite old and not available
681+
in some build environment (Airlock).
682+
https://github.com/googleapis/java-shared-config/issues/956
683+
Extracting this plugin declaration as a profile enables us to disable
684+
the CLIRR dependency resolution by `mvn -P=-clirr-compatibility-check ...`
685+
-->
686+
<activation>
687+
<!-- The compatibility check runs by default in CIs -->
688+
<jdk>[1.8,)</jdk>
689+
</activation>
690+
<build>
691+
<plugins>
692+
<plugin>
693+
<groupId>org.codehaus.mojo</groupId>
694+
<artifactId>clirr-maven-plugin</artifactId>
695+
<configuration>
696+
<ignoredDifferencesFile>clirr-ignored-differences.xml</ignoredDifferencesFile>
697+
<logResults>true</logResults>
698+
</configuration>
699+
<executions>
700+
<execution>
701+
<goals>
702+
<goal>check</goal>
703+
</goals>
704+
</execution>
705+
</executions>
706+
</plugin>
707+
</plugins>
708+
</build>
709+
</profile>
692710
</profiles>
693711
</project>

0 commit comments

Comments
 (0)