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/DefaultMavenExecutio
> nRequest.java
>
b/maven-core/src/main/java/org/apache/maven/execution/DefaultMavenExecuti
> onRequest.java index 3139846..09ead1a 100644
> ---
>
a/maven-core/src/main/java/org/apache/maven/execution/DefaultMavenExecutio
> nRequest.java +++
>
b/maven-core/src/main/java/org/apache/maven/execution/DefaultMavenExecutio
> nRequest.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/MavenExecutionReques
> t.java
>
b/maven-core/src/main/java/org/apache/maven/execution/MavenExecutionReque
> st.java index 5bd839a..cb4b207 100644
> ---
>
a/maven-core/src/main/java/org/apache/maven/execution/MavenExecutionReques
> t.java +++
>
b/maven-core/src/main/java/org/apache/maven/execution/MavenExecutionReques
> t.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 );
>
> }
>
> }