The following comment has been added to this issue:

     Author: Brett Porter
    Created: Wed, 24 Nov 2004 5:07 PM
       Body:
Yes, the plugin should have a way to install for a user. I have worked on this, 
but not yet been able to find the correct semantics.

I don't believe that plugin:install should be changed to do this immediately 
because it may break behaviour expected by someone.

We have someone working with geronimo at the moment that may be able to help 
with this.

---------------------------------------------------------------------
View this comment:
  http://jira.codehaus.org/browse/MPPLUGIN-30?page=comments#action_27139

---------------------------------------------------------------------
View the issue:
  http://jira.codehaus.org/browse/MPPLUGIN-30

Here is an overview of the issue:
---------------------------------------------------------------------
        Key: MPPLUGIN-30
    Summary: Plugins on a per-user basis
       Type: Bug

     Status: Open
   Priority: Major

 Original Estimate: Unknown
 Time Spent: Unknown
  Remaining: Unknown

    Project: maven-plugin-plugin
   Versions:
             1.5

   Assignee: Brett Porter
   Reporter: Rodrigo S. de Castro

    Created: Wed, 24 Nov 2004 8:52 AM
    Updated: Wed, 24 Nov 2004 5:07 PM
Environment: Linux (Debian), Maven 1.1

Description:
Problem:

When I try to compile Geronimo, it fails when trying to install its plugin into 
the $MAVEN_HOME directory, since it is a shared installation in 
/usr/local/maven-1.1. It does not install the plugins on a per-user basis to my 
maven local directory (~/.maven). Is this the intended behaviour?

Analysis:

In the org.apache.maven.plugin.PluginManager class, which is called for 
plugin:install-now, the plugin is installed in the user plugins dir, as we may 
check through the following code:

      if ( cache )
      {
           FileUtils.copyFileToDirectory( file, userPluginsDir );
           cacheManager.registerPlugin( pluginName, housing );
           housing.parse( cacheManager );
           cacheManager.saveCache( unpackedPluginsDir );
      }

Since I am not sure if the behaviour was intentional, I would like to know your 
opinion about that. 

>From the point of view that there is an inconsistent behaviour, I will attach 
>a patch that changes plugin:install to do the same as plugin:install-now: 
>install in the user directory. With this patch, current repository version of 
>Apache Geronimo works properly.

Concerning plugin removal, the code already check both directories (global and 
user), as you may check here (plugin/plugin.jelly):

    <define:tag name="uninstall">
      <ant:delete verbose="false" failonerror="false">
        <ant:fileset dir="${maven.plugin.dir}">
          <ant:include name="${name}-*.jar" />
        </ant:fileset>
        <ant:fileset dir="${maven.plugin.user.dir}">
          <ant:include name="${name}-*.jar" />
        </ant:fileset>
      </ant:delete>

Thank you!


---------------------------------------------------------------------
JIRA INFORMATION:
This message is automatically generated by JIRA.

If you think it was sent incorrectly contact one of the administrators:
   http://jira.codehaus.org/secure/Administrators.jspa

If you want more information on JIRA, or have a bug to report see:
   http://www.atlassian.com/software/jira


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to