diff --git a/maven-artifact/src/main/java/org/apache/maven/artifact/transform/SnapshotTransformation.java b/maven-artifact/src/main/java/org/apache/maven/artifact/transform/SnapshotTransformation.java index 9e95e6ef19..f03d8527cf 100644 --- a/maven-artifact/src/main/java/org/apache/maven/artifact/transform/SnapshotTransformation.java +++ b/maven-artifact/src/main/java/org/apache/maven/artifact/transform/SnapshotTransformation.java @@ -141,31 +141,26 @@ public class SnapshotTransformation String version = localMetadata.constructVersion(); - if ( getLogger().isInfoEnabled() && !alreadyResolved ) - { - if ( !version.equals( artifact.getBaseVersion() ) ) - { - String message = artifact.getArtifactId() + ": resolved to version " + version; - if ( artifact.getRepository() != null ) - { - message += " from repository " + artifact.getRepository().getId(); - } - else - { - message += " from local repository"; - } - getLogger().info( message ); - } - else - { - // Locally installed file is newer, don't use the resolved version - getLogger().info( artifact.getArtifactId() + ": using locally installed snapshot" ); - } - } - // TODO: if the POM and JAR are inconsistent, this might mean that different version of each are used if ( !artifact.getFile().exists() || localMetadata.newerThanFile( artifact.getFile() ) ) { + if ( getLogger().isInfoEnabled() && !alreadyResolved ) + { + if ( !version.equals( artifact.getBaseVersion() ) ) + { + String message = artifact.getArtifactId() + ": resolved to version " + version; + if ( artifact.getRepository() != null ) + { + message += " from repository " + artifact.getRepository().getId(); + } + else + { + message += " from local repository"; + } + getLogger().info( message ); + } + } + artifact.setVersion( version ); try { @@ -176,6 +171,14 @@ public class SnapshotTransformation throw new ArtifactMetadataRetrievalException( "Error reading local metadata", e ); } } + else + { + if ( getLogger().isInfoEnabled() && !alreadyResolved ) + { + // Locally installed file is newer, don't use the resolved version + getLogger().info( artifact.getArtifactId() + ": using locally installed snapshot" ); + } + } } }