Benjamin Bentmann wrote:

+            //create a temporary local repository with unique id
+ this.localRepository = new DefaultArtifactRepository( Long.toHexString( System.currentTimeMillis() ), "file://" + this.alternateLocalRepository.getAbsolutePath(), new DefaultRepositoryLayout() );

Directly using implementation classes instead of interfaces/components is bad with regard to future refactoring of Maven internals. Please use the ArtifactRepositoryFactory instead of new DefaultArtifactRepository().

For clarity, this applies to new DefaultRepositoryLayout() as well, it's a component so please use @component to access it via dependency injection.


Benjamin

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to