You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@maven.apache.org by ah...@apache.org on 2005/09/22 16:25:00 UTC
svn commit: r290949 - /maven/maven-1/plugins/trunk/plugin/plugin.jelly
Author: aheritier
Date: Thu Sep 22 07:24:54 2005
New Revision: 290949
URL: http://svn.apache.org/viewcvs?rev=290949&view=rev
Log:
We force maven to load the plugin with maven:set if it's not already done.
I don't know why maven:get don't do it. Bug ? Feature ?
Modified:
maven/maven-1/plugins/trunk/plugin/plugin.jelly
Modified: maven/maven-1/plugins/trunk/plugin/plugin.jelly
URL: http://svn.apache.org/viewcvs/maven/maven-1/plugins/trunk/plugin/plugin.jelly?rev=290949&r1=290948&r2=290949&view=diff
==============================================================================
--- maven/maven-1/plugins/trunk/plugin/plugin.jelly (original)
+++ maven/maven-1/plugins/trunk/plugin/plugin.jelly Thu Sep 22 07:24:54 2005
@@ -126,6 +126,9 @@
</j:otherwise>
</j:choose>
+ <!-- A workaround to force maven to load the plugin if it's not already done -->
+ <maven:set plugin="${pluginToTest}" property="myFakePropertyUsedToLoadThePlugin" value="XXXX" />
+ <!-- maven:get don't want to do it -->
<maven:get plugin="${pluginToTest}" property="plugin" var="installedPlugin" />
<!-- Check if the plugin is installed -->
<j:choose>
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org
RE: svn commit: r290949 - /maven/maven-1/plugins/trunk/plugin/plugin.jelly
Posted by Arnaud HERITIER <ah...@gmail.com>.
Ok, it's what I saw in an issue.
I found his mail. I'll take a look at it.
Cheers
Arnaud
>
> Bug that can't be fixed without breaking backwards
> compatibility. Check out Andy Glick's post to the dev list
> recently - I haven't reviewed it yet, but it was meant to
> document this.
>
> - Brett
>
> aheritier@apache.org wrote:
>
> >Author: aheritier
> >Date: Thu Sep 22 07:24:54 2005
> >New Revision: 290949
> >
> >URL: http://svn.apache.org/viewcvs?rev=290949&view=rev
> >Log:
> >We force maven to load the plugin with maven:set if it's not
> already done.
> >I don't know why maven:get don't do it. Bug ? Feature ?
> >
> >Modified:
> > maven/maven-1/plugins/trunk/plugin/plugin.jelly
> >
> >Modified: maven/maven-1/plugins/trunk/plugin/plugin.jelly
> >URL:
> >http://svn.apache.org/viewcvs/maven/maven-1/plugins/trunk/plu
> gin/plugin
> >.jelly?rev=290949&r1=290948&r2=290949&view=diff
> >=============================================================
> ==========
> >=======
> >--- maven/maven-1/plugins/trunk/plugin/plugin.jelly (original)
> >+++ maven/maven-1/plugins/trunk/plugin/plugin.jelly Thu Sep
> 22 07:24:54
> >+++ 2005
> >@@ -126,6 +126,9 @@
> > </j:otherwise>
> > </j:choose>
> >
> >+ <!-- A workaround to force maven to load the plugin if
> it's not already done -->
> >+ <maven:set plugin="${pluginToTest}"
> property="myFakePropertyUsedToLoadThePlugin" value="XXXX" />
> >+ <!-- maven:get don't want to do it -->
> > <maven:get plugin="${pluginToTest}" property="plugin"
> var="installedPlugin" />
> > <!-- Check if the plugin is installed -->
> > <j:choose>
> >
> >
> >
> >---------------------------------------------------------------------
> >To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org For
> additional
> >commands, e-mail: dev-help@maven.apache.org
> >
> >
> >
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org For
> additional commands, e-mail: dev-help@maven.apache.org
>
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org
Re: svn commit: r290949 - /maven/maven-1/plugins/trunk/plugin/plugin.jelly
Posted by Brett Porter <br...@apache.org>.
Bug that can't be fixed without breaking backwards compatibility. Check
out Andy Glick's post to the dev list recently - I haven't reviewed it
yet, but it was meant to document this.
- Brett
aheritier@apache.org wrote:
>Author: aheritier
>Date: Thu Sep 22 07:24:54 2005
>New Revision: 290949
>
>URL: http://svn.apache.org/viewcvs?rev=290949&view=rev
>Log:
>We force maven to load the plugin with maven:set if it's not already done.
>I don't know why maven:get don't do it. Bug ? Feature ?
>
>Modified:
> maven/maven-1/plugins/trunk/plugin/plugin.jelly
>
>Modified: maven/maven-1/plugins/trunk/plugin/plugin.jelly
>URL: http://svn.apache.org/viewcvs/maven/maven-1/plugins/trunk/plugin/plugin.jelly?rev=290949&r1=290948&r2=290949&view=diff
>==============================================================================
>--- maven/maven-1/plugins/trunk/plugin/plugin.jelly (original)
>+++ maven/maven-1/plugins/trunk/plugin/plugin.jelly Thu Sep 22 07:24:54 2005
>@@ -126,6 +126,9 @@
> </j:otherwise>
> </j:choose>
>
>+ <!-- A workaround to force maven to load the plugin if it's not already done -->
>+ <maven:set plugin="${pluginToTest}" property="myFakePropertyUsedToLoadThePlugin" value="XXXX" />
>+ <!-- maven:get don't want to do it -->
> <maven:get plugin="${pluginToTest}" property="plugin" var="installedPlugin" />
> <!-- Check if the plugin is installed -->
> <j:choose>
>
>
>
>---------------------------------------------------------------------
>To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
>For additional commands, e-mail: dev-help@maven.apache.org
>
>
>
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org