i'm on the fence about if this is good or not, but I think the option if provided should be simple-local-repository without the manager part. People already get confused about what's a local repo vs what's a repository manager and the mixing of these concepts here will make that worse.
On Sat, Feb 2, 2013 at 10:59 AM, <hbout...@apache.org> wrote: > Updated Branches: > refs/heads/master 71dd7f3d2 -> 5d06bc6a2 > > > fixed typo > > Project: http://git-wip-us.apache.org/repos/asf/maven/repo > Commit: http://git-wip-us.apache.org/repos/asf/maven/commit/5d06bc6a > Tree: http://git-wip-us.apache.org/repos/asf/maven/tree/5d06bc6a > Diff: http://git-wip-us.apache.org/repos/asf/maven/diff/5d06bc6a > > Branch: refs/heads/master > Commit: 5d06bc6a25d40da49b9f477e3c2b408505dbae61 > Parents: 71dd7f3 > Author: Hervé Boutemy <hbout...@apache.org> > Authored: Sat Feb 2 16:59:20 2013 +0100 > Committer: Hervé Boutemy <hbout...@apache.org> > Committed: Sat Feb 2 16:59:20 2013 +0100 > > ---------------------------------------------------------------------- > .../main/java/org/apache/maven/DefaultMaven.java | 2 +- > .../execution/DefaultMavenExecutionRequest.java | 10 +++++----- > .../maven/execution/MavenExecutionRequest.java | 4 ++-- > .../main/java/org/apache/maven/cli/CLIManager.java | 4 ++-- > .../main/java/org/apache/maven/cli/MavenCli.java | 2 +- > 5 files changed, 11 insertions(+), 11 deletions(-) > ---------------------------------------------------------------------- > > > > http://git-wip-us.apache.org/repos/asf/maven/blob/5d06bc6a/maven-core/src/main/java/org/apache/maven/DefaultMaven.java > ---------------------------------------------------------------------- > diff --git a/maven-core/src/main/java/org/apache/maven/DefaultMaven.java > b/maven-core/src/main/java/org/apache/maven/DefaultMaven.java > index d85f1ac..ac92afc 100644 > --- a/maven-core/src/main/java/org/apache/maven/DefaultMaven.java > +++ b/maven-core/src/main/java/org/apache/maven/DefaultMaven.java > @@ -358,7 +358,7 @@ public class DefaultMaven > > LocalRepository localRepo = new LocalRepository( > request.getLocalRepository().getBasedir() ); > > - if ( request.isUseSimpleLocalRepostoryManager() ) > + if ( request.isUseSimpleLocalRepositoryManager() ) > { > try > { > > > http://git-wip-us.apache.org/repos/asf/maven/blob/5d06bc6a/maven-core/src/main/java/org/apache/maven/execution/DefaultMavenExecutionRequest.java > ---------------------------------------------------------------------- > diff --git > a/maven-core/src/main/java/org/apache/maven/execution/DefaultMavenExecutionRequest.java > b/maven-core/src/main/java/org/apache/maven/execution/DefaultMavenExecutionRequest.java > index 3139846..09ead1a 100644 > --- > a/maven-core/src/main/java/org/apache/maven/execution/DefaultMavenExecutionRequest.java > +++ > b/maven-core/src/main/java/org/apache/maven/execution/DefaultMavenExecutionRequest.java > @@ -143,7 +143,7 @@ public class DefaultMavenExecutionRequest > */ > private boolean noSnapshotUpdates; > > - private boolean useSimpleLocalRepostoryManager = false; > + private boolean useSimpleLocalRepositoryManager = false; > > public DefaultMavenExecutionRequest() > { > @@ -1078,14 +1078,14 @@ public class DefaultMavenExecutionRequest > return this; > } > > - public boolean isUseSimpleLocalRepostoryManager() > + public boolean isUseSimpleLocalRepositoryManager() > { > - return this.useSimpleLocalRepostoryManager; > + return this.useSimpleLocalRepositoryManager; > } > > - public MavenExecutionRequest setUseSimpleLocalRepostoryManager( > boolean useSimpleLocalRepostoryManager ) > + public MavenExecutionRequest setUseSimpleLocalRepositoryManager( > boolean useSimpleLocalRepositoryManager ) > { > - this.useSimpleLocalRepostoryManager = > useSimpleLocalRepostoryManager; > + this.useSimpleLocalRepositoryManager = > useSimpleLocalRepositoryManager; > return this; > } > } > > > http://git-wip-us.apache.org/repos/asf/maven/blob/5d06bc6a/maven-core/src/main/java/org/apache/maven/execution/MavenExecutionRequest.java > ---------------------------------------------------------------------- > diff --git > a/maven-core/src/main/java/org/apache/maven/execution/MavenExecutionRequest.java > b/maven-core/src/main/java/org/apache/maven/execution/MavenExecutionRequest.java > index 5bd839a..cb4b207 100644 > --- > a/maven-core/src/main/java/org/apache/maven/execution/MavenExecutionRequest.java > +++ > b/maven-core/src/main/java/org/apache/maven/execution/MavenExecutionRequest.java > @@ -286,11 +286,11 @@ public interface MavenExecutionRequest > /** > * @since 3.1 > */ > - boolean isUseSimpleLocalRepostoryManager(); > + boolean isUseSimpleLocalRepositoryManager(); > > /** > * @since 3.1 > */ > - MavenExecutionRequest setUseSimpleLocalRepostoryManager( boolean > useSimpleLocalRepostoryManager ); > + MavenExecutionRequest setUseSimpleLocalRepositoryManager( boolean > useSimpleLocalRepositoryManager ); > > } > > > http://git-wip-us.apache.org/repos/asf/maven/blob/5d06bc6a/maven-embedder/src/main/java/org/apache/maven/cli/CLIManager.java > ---------------------------------------------------------------------- > diff --git > a/maven-embedder/src/main/java/org/apache/maven/cli/CLIManager.java > b/maven-embedder/src/main/java/org/apache/maven/cli/CLIManager.java > index 44eb7a3..4bcbda6 100644 > --- a/maven-embedder/src/main/java/org/apache/maven/cli/CLIManager.java > +++ b/maven-embedder/src/main/java/org/apache/maven/cli/CLIManager.java > @@ -137,10 +137,10 @@ public class CLIManager > options.addOption( OptionBuilder.withLongOpt( "threads" > ).hasArg().withDescription( "Thread count, for instance 2.0C where C is > core multiplied" ).create( THREADS ) ); > > options.addOption( OptionBuilder.withLongOpt( > "simple-local-repository-manager" ).withDescription( > - "Use a simple local Repository Manager (no use of > _maven.repositories) can be activated using > -Dmaven.simpleLocalRepoMan=true" ).create( > + "Use a simple Local Repository Manager, ie no use of > _maven.repositories. Can be activated using > -Dmaven.simpleLocalRepoMan=true" ).create( > SIMPLE_LOCAL_REPOSITORY_MANAGER ) ); > - // Adding this back in for compatibility with the verifier that > hard codes this option. > > + // Adding this back in for compatibility with the verifier that > hard codes this option. > options.addOption( OptionBuilder.withLongOpt( > "no-plugin-registry" ).withDescription( "Ineffective, only kept for > backward compatibility" ).create( "npr" ) ); > options.addOption( OptionBuilder.withLongOpt( > "check-plugin-updates" ).withDescription( "Ineffective, only kept for > backward compatibility" ).create( "cpu" ) ); > options.addOption( OptionBuilder.withLongOpt( "update-plugins" > ).withDescription( "Ineffective, only kept for backward compatibility" > ).create( "up" ) ); > > > http://git-wip-us.apache.org/repos/asf/maven/blob/5d06bc6a/maven-embedder/src/main/java/org/apache/maven/cli/MavenCli.java > ---------------------------------------------------------------------- > diff --git > a/maven-embedder/src/main/java/org/apache/maven/cli/MavenCli.java > b/maven-embedder/src/main/java/org/apache/maven/cli/MavenCli.java > index a6ce497..9fad9e6 100644 > --- a/maven-embedder/src/main/java/org/apache/maven/cli/MavenCli.java > +++ b/maven-embedder/src/main/java/org/apache/maven/cli/MavenCli.java > @@ -544,7 +544,7 @@ public class MavenCli > { > if ( cliRequest.commandLine.hasOption( > CLIManager.SIMPLE_LOCAL_REPOSITORY_MANAGER ) || Boolean.getBoolean( > "maven.simpleLocalRepoMan" ) ) > { > - cliRequest.request.setUseSimpleLocalRepostoryManager( true ); > + cliRequest.request.setUseSimpleLocalRepositoryManager( true ); > } > } > > >
