You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by "Benjamin Bentmann (JIRA)" <ji...@codehaus.org> on 2011/08/05 11:27:43 UTC
[jira] Closed: (MNG-5149)
DefaultArtifactRepositoryFactory.createArtifactRepository overload w/
layoutId passes wrong (repo) id
[ https://jira.codehaus.org/browse/MNG-5149?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Benjamin Bentmann closed MNG-5149.
----------------------------------
Resolution: Fixed
Fix Version/s: 3.0.4
Assignee: Benjamin Bentmann
Good catch, fixed in [1154138|http://svn.apache.org/viewvc?view=revision&revision=1154138].
> DefaultArtifactRepositoryFactory.createArtifactRepository overload w/ layoutId passes wrong (repo) id
> -----------------------------------------------------------------------------------------------------
>
> Key: MNG-5149
> URL: https://jira.codehaus.org/browse/MNG-5149
> Project: Maven 2 & 3
> Issue Type: Bug
> Components: Embedding
> Affects Versions: 3.0.3
> Reporter: Jesse Glick
> Assignee: Benjamin Bentmann
> Fix For: 3.0.4
>
>
> In {[DefaultArtifactRepositoryFactory.java}} there is a method overload:
> {noformat}
> public ArtifactRepository createArtifactRepository( String id, String url, String layoutId,
> ArtifactRepositoryPolicy snapshots,
> ArtifactRepositoryPolicy releases )
> throws UnknownRepositoryLayoutException
> {
> return injectSession( factory.createArtifactRepository( layoutId, url, layoutId, snapshots, releases ), true );
> }
> {noformat}
> Note that {{layoutId}} is used twice and {{id}} not at all. The result is that you get a remote repo apparently with an ID of {{default}}, rather than e.g. {{central}} as you intended.
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira