SONAR-2849 Remove declarations of third-party repositories from root pom

Enforce that repositories not declared in pom.xml, except of modules
sonar-checkstyle-plugin and sonar-squid-java-plugin due to dependency
on Checktyle 5.1, which is not available in Central.

This allows to remove side effects in Maven plugins.
This commit is contained in:
Evgeny Mandrikov 2011-10-01 05:14:56 +04:00
parent af58e35a58
commit eb20690f5b
3 changed files with 72 additions and 15 deletions

View File

@ -17,6 +17,21 @@
<checkstyle.version>5.1</checkstyle.version>
</properties>
<repositories>
<!-- for checkstyle 5.1 -->
<repository>
<id>sonar</id>
<name>Sonar</name>
<url>http://repository.sonarsource.org/content/repositories/sonar</url>
<releases>
<enabled>true</enabled>
</releases>
<snapshots>
<enabled>false</enabled>
</snapshots>
</repository>
</repositories>
<dependencies>
<dependency>
<groupId>org.codehaus.sonar</groupId>
@ -90,6 +105,20 @@
</testResource>
</testResources>
<plugins>
<plugin>
<!-- TODO remove (see SONAR-2938) -->
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-enforcer-plugin</artifactId>
<executions>
<execution>
<id>enforce-no-repositories</id>
<configuration>
<skip>true</skip>
</configuration>
</execution>
</executions>
</plugin>
<plugin>
<groupId>org.codehaus.sonar</groupId>
<artifactId>sonar-packaging-maven-plugin</artifactId>

View File

@ -13,6 +13,21 @@
<name>Sonar :: Plugins :: Squid Java</name>
<description>Squid analyzer for Java.</description>
<repositories>
<!-- for checkstyle 5.1 -->
<repository>
<id>sonar</id>
<name>Sonar</name>
<url>http://repository.sonarsource.org/content/repositories/sonar</url>
<releases>
<enabled>true</enabled>
</releases>
<snapshots>
<enabled>false</enabled>
</snapshots>
</repository>
</repositories>
<dependencies>
<dependency>
<groupId>org.codehaus.sonar</groupId>
@ -85,6 +100,20 @@
<build>
<plugins>
<plugin>
<!-- TODO remove (see SONAR-2938) -->
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-enforcer-plugin</artifactId>
<executions>
<execution>
<id>enforce-no-repositories</id>
<configuration>
<skip>true</skip>
</configuration>
</execution>
</executions>
</plugin>
<plugin>
<groupId>org.codehaus.sonar</groupId>
<artifactId>sonar-packaging-maven-plugin</artifactId>

29
pom.xml
View File

@ -74,21 +74,6 @@
</snapshotRepository>
</distributionManagement>
<repositories>
<repository>
<!-- for checkstyle 5.1-->
<id>sonar</id>
<name>Sonar</name>
<url>http://repository.sonarsource.org/content/repositories/sonar</url>
<releases>
<enabled>true</enabled>
</releases>
<snapshots>
<enabled>false</enabled>
</snapshots>
</repository>
</repositories>
<properties>
<!--
Warning before upgrading Derby to 10.8 : new conversion from BOOLEAN to CHAR.
@ -370,6 +355,20 @@
</rules>
</configuration>
</execution>
<execution>
<id>enforce-no-repositories</id>
<goals>
<goal>enforce</goal>
</goals>
<configuration>
<rules>
<requireNoRepositories>
<!-- Also see SONAR-2849 -->
<message>The rules for repo1.maven.org are that pom.xml files should not include repository definitions.</message>
</requireNoRepositories>
</rules>
</configuration>
</execution>
</executions>
</plugin>
<plugin>