Hi,

I have my doubts if this should be exposed as a mvn argument. This would also mean that we cannot remove it in the future. It would also suggest that it is a valid solution, but in fact it makes your build more unreliable. Users hitting this issue have often enough Maven knowledge to discover this option, so I don't see the need for this mvn commandline argument. Instead I would only use the MAVEN_OPTS option, and rename it to LegacyLocalRepository instead of SimpleLocalRepository to encourage not to use it.

WDYT?

Robert


Op Tue, 05 Feb 2013 00:02:47 +0100 schreef Hervé BOUTEMY <herve.bout...@free.fr>:

good idea

any objection?

Regards,

Hervé

Le lundi 4 février 2013 11:11:32 Brian Fox a écrit :
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 );
>
>          }
>
>      }

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
For additional commands, e-mail: dev-h...@maven.apache.org

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
For additional commands, e-mail: dev-h...@maven.apache.org

Reply via email to