You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomee.apache.org by David Blevins <da...@gmail.com> on 2013/05/09 19:19:58 UTC
Coordinates in Provisioning (was Re: svn commit: r1480535)
Seems like there's a simple solution, let's not use 'mvn:' as the prefix if we aren't using the Maven coordinate system.
Any ideas on a better prefix? 'karaf:' or 'aether:'
As well, do you have any reference to Aether that shows this alternate coordinates?
-David
On May 9, 2013, at 12:38 AM, Romain Manni-Bucau <rm...@gmail.com> wrote:
> Hi
>
> That's normal:
> - matches better default case
> - consistent with karaf/aether
> ---------- Message transféré ----------
> De : <db...@apache.org>
> Date : 9 mai 2013 08:30
> Objet : svn commit: r1480535 -
> /tomee/tomee/trunk/utils/openejb-provisionning/src/main/java/org/apache/openejb/resolver/maven/Parser.java
> À : <co...@tomee.apache.org>
>
> Author: dblevins
> Date: Thu May 9 06:30:22 2013
> New Revision: 1480535
>
> URL: http://svn.apache.org/r1480535
> Log:
> Doesn't seem like the coordinates used in this chunk of code match the
> maven coordinate system.
>
> Modified:
>
> tomee/tomee/trunk/utils/openejb-provisionning/src/main/java/org/apache/openejb/resolver/maven/Parser.java
>
> Modified:
> tomee/tomee/trunk/utils/openejb-provisionning/src/main/java/org/apache/openejb/resolver/maven/Parser.java
> URL:
> http://svn.apache.org/viewvc/tomee/tomee/trunk/utils/openejb-provisionning/src/main/java/org/apache/openejb/resolver/maven/Parser.java?rev=1480535&r1=1480534&r2=1480535&view=diff
> ==============================================================================
> ---
> tomee/tomee/trunk/utils/openejb-provisionning/src/main/java/org/apache/openejb/resolver/maven/Parser.java
> (original)
> +++
> tomee/tomee/trunk/utils/openejb-provisionning/src/main/java/org/apache/openejb/resolver/maven/Parser.java
> Thu May 9 06:30:22 2013
> @@ -19,6 +19,22 @@ package org.apache.openejb.resolver.mave
> import java.net.MalformedURLException;
> import org.ops4j.pax.url.maven.commons.MavenRepositoryURL;
>
> +/**
> + * This class doesn't seem compliant with the documented Maven Coordinates
> of
> + *
> + * - groupId:artifactId:packaging:classifier:version
> + *
> + * http://maven.apache.org/pom.html#Maven_Coordinates
> + *
> + * For example, per the output of `mvn dependency:tree` TomEE is:
> + *
> + * - org.apache.openejb:apache-tomee:zip:webprofile:1.6.0-SNAPSHOT
> + *
> + * It instead seems to use its own non-maven coordinates of
> + *
> + * - groupId:artifactId:version:packaging:classifier
> + *
> + */
> public class Parser {
> public static final String VERSION_LATEST = "LATEST";
> private static final String SYNTAX =
> "mvn:[repository_url!]groupId/artifactId[/[version]/[type]]";
Re: Coordinates in Provisioning (was Re: svn commit: r1480535)
Posted by Romain Manni-Bucau <rm...@gmail.com>.
that's perfectly fine
the idea was to align with people using explicitely these coordinates (=
karaf, smix...)
maven guys doesnt use it at all
and these format support a repo too in the url which is not the case of
mvn, so far more useful in practise
We can still do a re-try strategy moving the version at the end if the
first resolution fails but we need to keep current behavior
*Romain Manni-Bucau*
*Twitter: @rmannibucau <https://twitter.com/rmannibucau>*
*Blog: **http://rmannibucau.wordpress.com/*<http://rmannibucau.wordpress.com/>
*LinkedIn: **http://fr.linkedin.com/in/rmannibucau*
*Github: https://github.com/rmannibucau*
2013/5/9 David Blevins <da...@gmail.com>
> Right, Aether is using the standard Maven coordinates I mention. So far
> we can't parse these.
>
> -David
>
> On May 9, 2013, at 1:23 PM, Romain Manni-Bucau <rm...@gmail.com>
> wrote:
>
> > aether just move the version at the end:
> >
> http://grepcode.com/file/repo1.maven.org/maven2/org.sonatype.aether/aether-util/1.13.1/org/sonatype/aether/util/artifact/DefaultArtifact.java?av=h#DefaultArtifact
> >
> > the point is the default usage is g:a:v + we forked pax-url-aether to do
> > the provisioning module
> >
> > we rely on it too in jarstxt-maven-plugin (but since it is only relevant
> > for jars that's not a big deal)
> >
> > *Romain Manni-Bucau*
> > *Twitter: @rmannibucau <https://twitter.com/rmannibucau>*
> > *Blog: **http://rmannibucau.wordpress.com/*<
> http://rmannibucau.wordpress.com/>
> > *LinkedIn: **http://fr.linkedin.com/in/rmannibucau*
> > *Github: https://github.com/rmannibucau*
> >
> >
> >
> > 2013/5/9 David Blevins <da...@gmail.com>
> >
> >> Looks to be a Karaf reference (OPS4j).
> >>
> >> You mentioned Aether. Any info on that?
> >>
> >> -David
> >>
> >> On May 9, 2013, at 12:54 PM, Romain Manni-Bucau <rm...@gmail.com>
> >> wrote:
> >>
> >>> https://ops4j1.jira.com/wiki/display/paxurl/Mvn+Protocol
> >>> Le 9 mai 2013 21:38, "David Blevins" <da...@gmail.com> a
> écrit :
> >>>
> >>>> Do you have any reference to Aether that shows the alternate
> >> coordinates?
> >>>>
> >>>>
> >>>> -David
> >>>>
> >>>>
> >>>> On May 9, 2013, at 10:37 AM, Romain Manni-Bucau <
> rmannibucau@gmail.com>
> >>>> wrote:
> >>>>
> >>>>> -1 for karaf,
> >>>>> -1 for aether
> >>>>>
> >>>>> +0 for aether-mvn
> >>>>>
> >>>>> that said that's too late David to change it (either order of info or
> >> the
> >>>>> name), we already released it several times
> >>>>>
> >>>>> *Romain Manni-Bucau*
> >>>>> *Twitter: @rmannibucau <https://twitter.com/rmannibucau>*
> >>>>> *Blog: **http://rmannibucau.wordpress.com/*<
> >>>> http://rmannibucau.wordpress.com/>
> >>>>> *LinkedIn: **http://fr.linkedin.com/in/rmannibucau*
> >>>>> *Github: https://github.com/rmannibucau*
> >>>>>
> >>>>>
> >>>>>
> >>>>> 2013/5/9 David Blevins <da...@gmail.com>
> >>>>>
> >>>>>> Seems like there's a simple solution, let's not use 'mvn:' as the
> >> prefix
> >>>>>> if we aren't using the Maven coordinate system.
> >>>>>>
> >>>>>> Any ideas on a better prefix? 'karaf:' or 'aether:'
> >>>>>>
> >>>>>> As well, do you have any reference to Aether that shows this
> alternate
> >>>>>> coordinates?
> >>>>>>
> >>>>>>
> >>>>>> -David
> >>>>>>
> >>>>>> On May 9, 2013, at 12:38 AM, Romain Manni-Bucau <
> >> rmannibucau@gmail.com>
> >>>>>> wrote:
> >>>>>>
> >>>>>>> Hi
> >>>>>>>
> >>>>>>> That's normal:
> >>>>>>> - matches better default case
> >>>>>>> - consistent with karaf/aether
> >>>>>>> ---------- Message transféré ----------
> >>>>>>> De : <db...@apache.org>
> >>>>>>> Date : 9 mai 2013 08:30
> >>>>>>> Objet : svn commit: r1480535 -
> >>>>>>>
> >>>>>>
> >>>>
> >>
> /tomee/tomee/trunk/utils/openejb-provisionning/src/main/java/org/apache/openejb/resolver/maven/Parser.java
> >>>>>>> À : <co...@tomee.apache.org>
> >>>>>>>
> >>>>>>> Author: dblevins
> >>>>>>> Date: Thu May 9 06:30:22 2013
> >>>>>>> New Revision: 1480535
> >>>>>>>
> >>>>>>> URL: http://svn.apache.org/r1480535
> >>>>>>> Log:
> >>>>>>> Doesn't seem like the coordinates used in this chunk of code match
> >> the
> >>>>>>> maven coordinate system.
> >>>>>>>
> >>>>>>> Modified:
> >>>>>>>
> >>>>>>>
> >>>>>>
> >>>>
> >>
> tomee/tomee/trunk/utils/openejb-provisionning/src/main/java/org/apache/openejb/resolver/maven/Parser.java
> >>>>>>>
> >>>>>>> Modified:
> >>>>>>>
> >>>>>>
> >>>>
> >>
> tomee/tomee/trunk/utils/openejb-provisionning/src/main/java/org/apache/openejb/resolver/maven/Parser.java
> >>>>>>> URL:
> >>>>>>>
> >>>>>>
> >>>>
> >>
> http://svn.apache.org/viewvc/tomee/tomee/trunk/utils/openejb-provisionning/src/main/java/org/apache/openejb/resolver/maven/Parser.java?rev=1480535&r1=1480534&r2=1480535&view=diff
> >>>>>>>
> >>>>>>
> >>>>
> >>
> ==============================================================================
> >>>>>>> ---
> >>>>>>>
> >>>>>>
> >>>>
> >>
> tomee/tomee/trunk/utils/openejb-provisionning/src/main/java/org/apache/openejb/resolver/maven/Parser.java
> >>>>>>> (original)
> >>>>>>> +++
> >>>>>>>
> >>>>>>
> >>>>
> >>
> tomee/tomee/trunk/utils/openejb-provisionning/src/main/java/org/apache/openejb/resolver/maven/Parser.java
> >>>>>>> Thu May 9 06:30:22 2013
> >>>>>>> @@ -19,6 +19,22 @@ package org.apache.openejb.resolver.mave
> >>>>>>> import java.net.MalformedURLException;
> >>>>>>> import org.ops4j.pax.url.maven.commons.MavenRepositoryURL;
> >>>>>>>
> >>>>>>> +/**
> >>>>>>> + * This class doesn't seem compliant with the documented Maven
> >>>>>> Coordinates
> >>>>>>> of
> >>>>>>> + *
> >>>>>>> + * - groupId:artifactId:packaging:classifier:version
> >>>>>>> + *
> >>>>>>> + * http://maven.apache.org/pom.html#Maven_Coordinates
> >>>>>>> + *
> >>>>>>> + * For example, per the output of `mvn dependency:tree` TomEE is:
> >>>>>>> + *
> >>>>>>> + * -
> org.apache.openejb:apache-tomee:zip:webprofile:1.6.0-SNAPSHOT
> >>>>>>> + *
> >>>>>>> + * It instead seems to use its own non-maven coordinates of
> >>>>>>> + *
> >>>>>>> + * - groupId:artifactId:version:packaging:classifier
> >>>>>>> + *
> >>>>>>> + */
> >>>>>>> public class Parser {
> >>>>>>> public static final String VERSION_LATEST = "LATEST";
> >>>>>>> private static final String SYNTAX =
> >>>>>>> "mvn:[repository_url!]groupId/artifactId[/[version]/[type]]";
> >>>>>>
> >>>>>>
> >>>>
> >>>>
> >>
> >>
>
>
Re: Coordinates in Provisioning (was Re: svn commit: r1480535)
Posted by David Blevins <da...@gmail.com>.
Right, Aether is using the standard Maven coordinates I mention. So far we can't parse these.
-David
On May 9, 2013, at 1:23 PM, Romain Manni-Bucau <rm...@gmail.com> wrote:
> aether just move the version at the end:
> http://grepcode.com/file/repo1.maven.org/maven2/org.sonatype.aether/aether-util/1.13.1/org/sonatype/aether/util/artifact/DefaultArtifact.java?av=h#DefaultArtifact
>
> the point is the default usage is g:a:v + we forked pax-url-aether to do
> the provisioning module
>
> we rely on it too in jarstxt-maven-plugin (but since it is only relevant
> for jars that's not a big deal)
>
> *Romain Manni-Bucau*
> *Twitter: @rmannibucau <https://twitter.com/rmannibucau>*
> *Blog: **http://rmannibucau.wordpress.com/*<http://rmannibucau.wordpress.com/>
> *LinkedIn: **http://fr.linkedin.com/in/rmannibucau*
> *Github: https://github.com/rmannibucau*
>
>
>
> 2013/5/9 David Blevins <da...@gmail.com>
>
>> Looks to be a Karaf reference (OPS4j).
>>
>> You mentioned Aether. Any info on that?
>>
>> -David
>>
>> On May 9, 2013, at 12:54 PM, Romain Manni-Bucau <rm...@gmail.com>
>> wrote:
>>
>>> https://ops4j1.jira.com/wiki/display/paxurl/Mvn+Protocol
>>> Le 9 mai 2013 21:38, "David Blevins" <da...@gmail.com> a écrit :
>>>
>>>> Do you have any reference to Aether that shows the alternate
>> coordinates?
>>>>
>>>>
>>>> -David
>>>>
>>>>
>>>> On May 9, 2013, at 10:37 AM, Romain Manni-Bucau <rm...@gmail.com>
>>>> wrote:
>>>>
>>>>> -1 for karaf,
>>>>> -1 for aether
>>>>>
>>>>> +0 for aether-mvn
>>>>>
>>>>> that said that's too late David to change it (either order of info or
>> the
>>>>> name), we already released it several times
>>>>>
>>>>> *Romain Manni-Bucau*
>>>>> *Twitter: @rmannibucau <https://twitter.com/rmannibucau>*
>>>>> *Blog: **http://rmannibucau.wordpress.com/*<
>>>> http://rmannibucau.wordpress.com/>
>>>>> *LinkedIn: **http://fr.linkedin.com/in/rmannibucau*
>>>>> *Github: https://github.com/rmannibucau*
>>>>>
>>>>>
>>>>>
>>>>> 2013/5/9 David Blevins <da...@gmail.com>
>>>>>
>>>>>> Seems like there's a simple solution, let's not use 'mvn:' as the
>> prefix
>>>>>> if we aren't using the Maven coordinate system.
>>>>>>
>>>>>> Any ideas on a better prefix? 'karaf:' or 'aether:'
>>>>>>
>>>>>> As well, do you have any reference to Aether that shows this alternate
>>>>>> coordinates?
>>>>>>
>>>>>>
>>>>>> -David
>>>>>>
>>>>>> On May 9, 2013, at 12:38 AM, Romain Manni-Bucau <
>> rmannibucau@gmail.com>
>>>>>> wrote:
>>>>>>
>>>>>>> Hi
>>>>>>>
>>>>>>> That's normal:
>>>>>>> - matches better default case
>>>>>>> - consistent with karaf/aether
>>>>>>> ---------- Message transféré ----------
>>>>>>> De : <db...@apache.org>
>>>>>>> Date : 9 mai 2013 08:30
>>>>>>> Objet : svn commit: r1480535 -
>>>>>>>
>>>>>>
>>>>
>> /tomee/tomee/trunk/utils/openejb-provisionning/src/main/java/org/apache/openejb/resolver/maven/Parser.java
>>>>>>> À : <co...@tomee.apache.org>
>>>>>>>
>>>>>>> Author: dblevins
>>>>>>> Date: Thu May 9 06:30:22 2013
>>>>>>> New Revision: 1480535
>>>>>>>
>>>>>>> URL: http://svn.apache.org/r1480535
>>>>>>> Log:
>>>>>>> Doesn't seem like the coordinates used in this chunk of code match
>> the
>>>>>>> maven coordinate system.
>>>>>>>
>>>>>>> Modified:
>>>>>>>
>>>>>>>
>>>>>>
>>>>
>> tomee/tomee/trunk/utils/openejb-provisionning/src/main/java/org/apache/openejb/resolver/maven/Parser.java
>>>>>>>
>>>>>>> Modified:
>>>>>>>
>>>>>>
>>>>
>> tomee/tomee/trunk/utils/openejb-provisionning/src/main/java/org/apache/openejb/resolver/maven/Parser.java
>>>>>>> URL:
>>>>>>>
>>>>>>
>>>>
>> http://svn.apache.org/viewvc/tomee/tomee/trunk/utils/openejb-provisionning/src/main/java/org/apache/openejb/resolver/maven/Parser.java?rev=1480535&r1=1480534&r2=1480535&view=diff
>>>>>>>
>>>>>>
>>>>
>> ==============================================================================
>>>>>>> ---
>>>>>>>
>>>>>>
>>>>
>> tomee/tomee/trunk/utils/openejb-provisionning/src/main/java/org/apache/openejb/resolver/maven/Parser.java
>>>>>>> (original)
>>>>>>> +++
>>>>>>>
>>>>>>
>>>>
>> tomee/tomee/trunk/utils/openejb-provisionning/src/main/java/org/apache/openejb/resolver/maven/Parser.java
>>>>>>> Thu May 9 06:30:22 2013
>>>>>>> @@ -19,6 +19,22 @@ package org.apache.openejb.resolver.mave
>>>>>>> import java.net.MalformedURLException;
>>>>>>> import org.ops4j.pax.url.maven.commons.MavenRepositoryURL;
>>>>>>>
>>>>>>> +/**
>>>>>>> + * This class doesn't seem compliant with the documented Maven
>>>>>> Coordinates
>>>>>>> of
>>>>>>> + *
>>>>>>> + * - groupId:artifactId:packaging:classifier:version
>>>>>>> + *
>>>>>>> + * http://maven.apache.org/pom.html#Maven_Coordinates
>>>>>>> + *
>>>>>>> + * For example, per the output of `mvn dependency:tree` TomEE is:
>>>>>>> + *
>>>>>>> + * - org.apache.openejb:apache-tomee:zip:webprofile:1.6.0-SNAPSHOT
>>>>>>> + *
>>>>>>> + * It instead seems to use its own non-maven coordinates of
>>>>>>> + *
>>>>>>> + * - groupId:artifactId:version:packaging:classifier
>>>>>>> + *
>>>>>>> + */
>>>>>>> public class Parser {
>>>>>>> public static final String VERSION_LATEST = "LATEST";
>>>>>>> private static final String SYNTAX =
>>>>>>> "mvn:[repository_url!]groupId/artifactId[/[version]/[type]]";
>>>>>>
>>>>>>
>>>>
>>>>
>>
>>
Re: Coordinates in Provisioning (was Re: svn commit: r1480535)
Posted by Romain Manni-Bucau <rm...@gmail.com>.
aether just move the version at the end:
http://grepcode.com/file/repo1.maven.org/maven2/org.sonatype.aether/aether-util/1.13.1/org/sonatype/aether/util/artifact/DefaultArtifact.java?av=h#DefaultArtifact
the point is the default usage is g:a:v + we forked pax-url-aether to do
the provisioning module
we rely on it too in jarstxt-maven-plugin (but since it is only relevant
for jars that's not a big deal)
*Romain Manni-Bucau*
*Twitter: @rmannibucau <https://twitter.com/rmannibucau>*
*Blog: **http://rmannibucau.wordpress.com/*<http://rmannibucau.wordpress.com/>
*LinkedIn: **http://fr.linkedin.com/in/rmannibucau*
*Github: https://github.com/rmannibucau*
2013/5/9 David Blevins <da...@gmail.com>
> Looks to be a Karaf reference (OPS4j).
>
> You mentioned Aether. Any info on that?
>
> -David
>
> On May 9, 2013, at 12:54 PM, Romain Manni-Bucau <rm...@gmail.com>
> wrote:
>
> > https://ops4j1.jira.com/wiki/display/paxurl/Mvn+Protocol
> > Le 9 mai 2013 21:38, "David Blevins" <da...@gmail.com> a écrit :
> >
> >> Do you have any reference to Aether that shows the alternate
> coordinates?
> >>
> >>
> >> -David
> >>
> >>
> >> On May 9, 2013, at 10:37 AM, Romain Manni-Bucau <rm...@gmail.com>
> >> wrote:
> >>
> >>> -1 for karaf,
> >>> -1 for aether
> >>>
> >>> +0 for aether-mvn
> >>>
> >>> that said that's too late David to change it (either order of info or
> the
> >>> name), we already released it several times
> >>>
> >>> *Romain Manni-Bucau*
> >>> *Twitter: @rmannibucau <https://twitter.com/rmannibucau>*
> >>> *Blog: **http://rmannibucau.wordpress.com/*<
> >> http://rmannibucau.wordpress.com/>
> >>> *LinkedIn: **http://fr.linkedin.com/in/rmannibucau*
> >>> *Github: https://github.com/rmannibucau*
> >>>
> >>>
> >>>
> >>> 2013/5/9 David Blevins <da...@gmail.com>
> >>>
> >>>> Seems like there's a simple solution, let's not use 'mvn:' as the
> prefix
> >>>> if we aren't using the Maven coordinate system.
> >>>>
> >>>> Any ideas on a better prefix? 'karaf:' or 'aether:'
> >>>>
> >>>> As well, do you have any reference to Aether that shows this alternate
> >>>> coordinates?
> >>>>
> >>>>
> >>>> -David
> >>>>
> >>>> On May 9, 2013, at 12:38 AM, Romain Manni-Bucau <
> rmannibucau@gmail.com>
> >>>> wrote:
> >>>>
> >>>>> Hi
> >>>>>
> >>>>> That's normal:
> >>>>> - matches better default case
> >>>>> - consistent with karaf/aether
> >>>>> ---------- Message transféré ----------
> >>>>> De : <db...@apache.org>
> >>>>> Date : 9 mai 2013 08:30
> >>>>> Objet : svn commit: r1480535 -
> >>>>>
> >>>>
> >>
> /tomee/tomee/trunk/utils/openejb-provisionning/src/main/java/org/apache/openejb/resolver/maven/Parser.java
> >>>>> À : <co...@tomee.apache.org>
> >>>>>
> >>>>> Author: dblevins
> >>>>> Date: Thu May 9 06:30:22 2013
> >>>>> New Revision: 1480535
> >>>>>
> >>>>> URL: http://svn.apache.org/r1480535
> >>>>> Log:
> >>>>> Doesn't seem like the coordinates used in this chunk of code match
> the
> >>>>> maven coordinate system.
> >>>>>
> >>>>> Modified:
> >>>>>
> >>>>>
> >>>>
> >>
> tomee/tomee/trunk/utils/openejb-provisionning/src/main/java/org/apache/openejb/resolver/maven/Parser.java
> >>>>>
> >>>>> Modified:
> >>>>>
> >>>>
> >>
> tomee/tomee/trunk/utils/openejb-provisionning/src/main/java/org/apache/openejb/resolver/maven/Parser.java
> >>>>> URL:
> >>>>>
> >>>>
> >>
> http://svn.apache.org/viewvc/tomee/tomee/trunk/utils/openejb-provisionning/src/main/java/org/apache/openejb/resolver/maven/Parser.java?rev=1480535&r1=1480534&r2=1480535&view=diff
> >>>>>
> >>>>
> >>
> ==============================================================================
> >>>>> ---
> >>>>>
> >>>>
> >>
> tomee/tomee/trunk/utils/openejb-provisionning/src/main/java/org/apache/openejb/resolver/maven/Parser.java
> >>>>> (original)
> >>>>> +++
> >>>>>
> >>>>
> >>
> tomee/tomee/trunk/utils/openejb-provisionning/src/main/java/org/apache/openejb/resolver/maven/Parser.java
> >>>>> Thu May 9 06:30:22 2013
> >>>>> @@ -19,6 +19,22 @@ package org.apache.openejb.resolver.mave
> >>>>> import java.net.MalformedURLException;
> >>>>> import org.ops4j.pax.url.maven.commons.MavenRepositoryURL;
> >>>>>
> >>>>> +/**
> >>>>> + * This class doesn't seem compliant with the documented Maven
> >>>> Coordinates
> >>>>> of
> >>>>> + *
> >>>>> + * - groupId:artifactId:packaging:classifier:version
> >>>>> + *
> >>>>> + * http://maven.apache.org/pom.html#Maven_Coordinates
> >>>>> + *
> >>>>> + * For example, per the output of `mvn dependency:tree` TomEE is:
> >>>>> + *
> >>>>> + * - org.apache.openejb:apache-tomee:zip:webprofile:1.6.0-SNAPSHOT
> >>>>> + *
> >>>>> + * It instead seems to use its own non-maven coordinates of
> >>>>> + *
> >>>>> + * - groupId:artifactId:version:packaging:classifier
> >>>>> + *
> >>>>> + */
> >>>>> public class Parser {
> >>>>> public static final String VERSION_LATEST = "LATEST";
> >>>>> private static final String SYNTAX =
> >>>>> "mvn:[repository_url!]groupId/artifactId[/[version]/[type]]";
> >>>>
> >>>>
> >>
> >>
>
>
Re: Coordinates in Provisioning (was Re: svn commit: r1480535)
Posted by David Blevins <da...@gmail.com>.
Looks to be a Karaf reference (OPS4j).
You mentioned Aether. Any info on that?
-David
On May 9, 2013, at 12:54 PM, Romain Manni-Bucau <rm...@gmail.com> wrote:
> https://ops4j1.jira.com/wiki/display/paxurl/Mvn+Protocol
> Le 9 mai 2013 21:38, "David Blevins" <da...@gmail.com> a écrit :
>
>> Do you have any reference to Aether that shows the alternate coordinates?
>>
>>
>> -David
>>
>>
>> On May 9, 2013, at 10:37 AM, Romain Manni-Bucau <rm...@gmail.com>
>> wrote:
>>
>>> -1 for karaf,
>>> -1 for aether
>>>
>>> +0 for aether-mvn
>>>
>>> that said that's too late David to change it (either order of info or the
>>> name), we already released it several times
>>>
>>> *Romain Manni-Bucau*
>>> *Twitter: @rmannibucau <https://twitter.com/rmannibucau>*
>>> *Blog: **http://rmannibucau.wordpress.com/*<
>> http://rmannibucau.wordpress.com/>
>>> *LinkedIn: **http://fr.linkedin.com/in/rmannibucau*
>>> *Github: https://github.com/rmannibucau*
>>>
>>>
>>>
>>> 2013/5/9 David Blevins <da...@gmail.com>
>>>
>>>> Seems like there's a simple solution, let's not use 'mvn:' as the prefix
>>>> if we aren't using the Maven coordinate system.
>>>>
>>>> Any ideas on a better prefix? 'karaf:' or 'aether:'
>>>>
>>>> As well, do you have any reference to Aether that shows this alternate
>>>> coordinates?
>>>>
>>>>
>>>> -David
>>>>
>>>> On May 9, 2013, at 12:38 AM, Romain Manni-Bucau <rm...@gmail.com>
>>>> wrote:
>>>>
>>>>> Hi
>>>>>
>>>>> That's normal:
>>>>> - matches better default case
>>>>> - consistent with karaf/aether
>>>>> ---------- Message transféré ----------
>>>>> De : <db...@apache.org>
>>>>> Date : 9 mai 2013 08:30
>>>>> Objet : svn commit: r1480535 -
>>>>>
>>>>
>> /tomee/tomee/trunk/utils/openejb-provisionning/src/main/java/org/apache/openejb/resolver/maven/Parser.java
>>>>> À : <co...@tomee.apache.org>
>>>>>
>>>>> Author: dblevins
>>>>> Date: Thu May 9 06:30:22 2013
>>>>> New Revision: 1480535
>>>>>
>>>>> URL: http://svn.apache.org/r1480535
>>>>> Log:
>>>>> Doesn't seem like the coordinates used in this chunk of code match the
>>>>> maven coordinate system.
>>>>>
>>>>> Modified:
>>>>>
>>>>>
>>>>
>> tomee/tomee/trunk/utils/openejb-provisionning/src/main/java/org/apache/openejb/resolver/maven/Parser.java
>>>>>
>>>>> Modified:
>>>>>
>>>>
>> tomee/tomee/trunk/utils/openejb-provisionning/src/main/java/org/apache/openejb/resolver/maven/Parser.java
>>>>> URL:
>>>>>
>>>>
>> http://svn.apache.org/viewvc/tomee/tomee/trunk/utils/openejb-provisionning/src/main/java/org/apache/openejb/resolver/maven/Parser.java?rev=1480535&r1=1480534&r2=1480535&view=diff
>>>>>
>>>>
>> ==============================================================================
>>>>> ---
>>>>>
>>>>
>> tomee/tomee/trunk/utils/openejb-provisionning/src/main/java/org/apache/openejb/resolver/maven/Parser.java
>>>>> (original)
>>>>> +++
>>>>>
>>>>
>> tomee/tomee/trunk/utils/openejb-provisionning/src/main/java/org/apache/openejb/resolver/maven/Parser.java
>>>>> Thu May 9 06:30:22 2013
>>>>> @@ -19,6 +19,22 @@ package org.apache.openejb.resolver.mave
>>>>> import java.net.MalformedURLException;
>>>>> import org.ops4j.pax.url.maven.commons.MavenRepositoryURL;
>>>>>
>>>>> +/**
>>>>> + * This class doesn't seem compliant with the documented Maven
>>>> Coordinates
>>>>> of
>>>>> + *
>>>>> + * - groupId:artifactId:packaging:classifier:version
>>>>> + *
>>>>> + * http://maven.apache.org/pom.html#Maven_Coordinates
>>>>> + *
>>>>> + * For example, per the output of `mvn dependency:tree` TomEE is:
>>>>> + *
>>>>> + * - org.apache.openejb:apache-tomee:zip:webprofile:1.6.0-SNAPSHOT
>>>>> + *
>>>>> + * It instead seems to use its own non-maven coordinates of
>>>>> + *
>>>>> + * - groupId:artifactId:version:packaging:classifier
>>>>> + *
>>>>> + */
>>>>> public class Parser {
>>>>> public static final String VERSION_LATEST = "LATEST";
>>>>> private static final String SYNTAX =
>>>>> "mvn:[repository_url!]groupId/artifactId[/[version]/[type]]";
>>>>
>>>>
>>
>>
Re: Coordinates in Provisioning (was Re: svn commit: r1480535)
Posted by Romain Manni-Bucau <rm...@gmail.com>.
https://ops4j1.jira.com/wiki/display/paxurl/Mvn+Protocol
Le 9 mai 2013 21:38, "David Blevins" <da...@gmail.com> a écrit :
> Do you have any reference to Aether that shows the alternate coordinates?
>
>
> -David
>
>
> On May 9, 2013, at 10:37 AM, Romain Manni-Bucau <rm...@gmail.com>
> wrote:
>
> > -1 for karaf,
> > -1 for aether
> >
> > +0 for aether-mvn
> >
> > that said that's too late David to change it (either order of info or the
> > name), we already released it several times
> >
> > *Romain Manni-Bucau*
> > *Twitter: @rmannibucau <https://twitter.com/rmannibucau>*
> > *Blog: **http://rmannibucau.wordpress.com/*<
> http://rmannibucau.wordpress.com/>
> > *LinkedIn: **http://fr.linkedin.com/in/rmannibucau*
> > *Github: https://github.com/rmannibucau*
> >
> >
> >
> > 2013/5/9 David Blevins <da...@gmail.com>
> >
> >> Seems like there's a simple solution, let's not use 'mvn:' as the prefix
> >> if we aren't using the Maven coordinate system.
> >>
> >> Any ideas on a better prefix? 'karaf:' or 'aether:'
> >>
> >> As well, do you have any reference to Aether that shows this alternate
> >> coordinates?
> >>
> >>
> >> -David
> >>
> >> On May 9, 2013, at 12:38 AM, Romain Manni-Bucau <rm...@gmail.com>
> >> wrote:
> >>
> >>> Hi
> >>>
> >>> That's normal:
> >>> - matches better default case
> >>> - consistent with karaf/aether
> >>> ---------- Message transféré ----------
> >>> De : <db...@apache.org>
> >>> Date : 9 mai 2013 08:30
> >>> Objet : svn commit: r1480535 -
> >>>
> >>
> /tomee/tomee/trunk/utils/openejb-provisionning/src/main/java/org/apache/openejb/resolver/maven/Parser.java
> >>> À : <co...@tomee.apache.org>
> >>>
> >>> Author: dblevins
> >>> Date: Thu May 9 06:30:22 2013
> >>> New Revision: 1480535
> >>>
> >>> URL: http://svn.apache.org/r1480535
> >>> Log:
> >>> Doesn't seem like the coordinates used in this chunk of code match the
> >>> maven coordinate system.
> >>>
> >>> Modified:
> >>>
> >>>
> >>
> tomee/tomee/trunk/utils/openejb-provisionning/src/main/java/org/apache/openejb/resolver/maven/Parser.java
> >>>
> >>> Modified:
> >>>
> >>
> tomee/tomee/trunk/utils/openejb-provisionning/src/main/java/org/apache/openejb/resolver/maven/Parser.java
> >>> URL:
> >>>
> >>
> http://svn.apache.org/viewvc/tomee/tomee/trunk/utils/openejb-provisionning/src/main/java/org/apache/openejb/resolver/maven/Parser.java?rev=1480535&r1=1480534&r2=1480535&view=diff
> >>>
> >>
> ==============================================================================
> >>> ---
> >>>
> >>
> tomee/tomee/trunk/utils/openejb-provisionning/src/main/java/org/apache/openejb/resolver/maven/Parser.java
> >>> (original)
> >>> +++
> >>>
> >>
> tomee/tomee/trunk/utils/openejb-provisionning/src/main/java/org/apache/openejb/resolver/maven/Parser.java
> >>> Thu May 9 06:30:22 2013
> >>> @@ -19,6 +19,22 @@ package org.apache.openejb.resolver.mave
> >>> import java.net.MalformedURLException;
> >>> import org.ops4j.pax.url.maven.commons.MavenRepositoryURL;
> >>>
> >>> +/**
> >>> + * This class doesn't seem compliant with the documented Maven
> >> Coordinates
> >>> of
> >>> + *
> >>> + * - groupId:artifactId:packaging:classifier:version
> >>> + *
> >>> + * http://maven.apache.org/pom.html#Maven_Coordinates
> >>> + *
> >>> + * For example, per the output of `mvn dependency:tree` TomEE is:
> >>> + *
> >>> + * - org.apache.openejb:apache-tomee:zip:webprofile:1.6.0-SNAPSHOT
> >>> + *
> >>> + * It instead seems to use its own non-maven coordinates of
> >>> + *
> >>> + * - groupId:artifactId:version:packaging:classifier
> >>> + *
> >>> + */
> >>> public class Parser {
> >>> public static final String VERSION_LATEST = "LATEST";
> >>> private static final String SYNTAX =
> >>> "mvn:[repository_url!]groupId/artifactId[/[version]/[type]]";
> >>
> >>
>
>
Re: Coordinates in Provisioning (was Re: svn commit: r1480535)
Posted by David Blevins <da...@gmail.com>.
Do you have any reference to Aether that shows the alternate coordinates?
-David
On May 9, 2013, at 10:37 AM, Romain Manni-Bucau <rm...@gmail.com> wrote:
> -1 for karaf,
> -1 for aether
>
> +0 for aether-mvn
>
> that said that's too late David to change it (either order of info or the
> name), we already released it several times
>
> *Romain Manni-Bucau*
> *Twitter: @rmannibucau <https://twitter.com/rmannibucau>*
> *Blog: **http://rmannibucau.wordpress.com/*<http://rmannibucau.wordpress.com/>
> *LinkedIn: **http://fr.linkedin.com/in/rmannibucau*
> *Github: https://github.com/rmannibucau*
>
>
>
> 2013/5/9 David Blevins <da...@gmail.com>
>
>> Seems like there's a simple solution, let's not use 'mvn:' as the prefix
>> if we aren't using the Maven coordinate system.
>>
>> Any ideas on a better prefix? 'karaf:' or 'aether:'
>>
>> As well, do you have any reference to Aether that shows this alternate
>> coordinates?
>>
>>
>> -David
>>
>> On May 9, 2013, at 12:38 AM, Romain Manni-Bucau <rm...@gmail.com>
>> wrote:
>>
>>> Hi
>>>
>>> That's normal:
>>> - matches better default case
>>> - consistent with karaf/aether
>>> ---------- Message transféré ----------
>>> De : <db...@apache.org>
>>> Date : 9 mai 2013 08:30
>>> Objet : svn commit: r1480535 -
>>>
>> /tomee/tomee/trunk/utils/openejb-provisionning/src/main/java/org/apache/openejb/resolver/maven/Parser.java
>>> À : <co...@tomee.apache.org>
>>>
>>> Author: dblevins
>>> Date: Thu May 9 06:30:22 2013
>>> New Revision: 1480535
>>>
>>> URL: http://svn.apache.org/r1480535
>>> Log:
>>> Doesn't seem like the coordinates used in this chunk of code match the
>>> maven coordinate system.
>>>
>>> Modified:
>>>
>>>
>> tomee/tomee/trunk/utils/openejb-provisionning/src/main/java/org/apache/openejb/resolver/maven/Parser.java
>>>
>>> Modified:
>>>
>> tomee/tomee/trunk/utils/openejb-provisionning/src/main/java/org/apache/openejb/resolver/maven/Parser.java
>>> URL:
>>>
>> http://svn.apache.org/viewvc/tomee/tomee/trunk/utils/openejb-provisionning/src/main/java/org/apache/openejb/resolver/maven/Parser.java?rev=1480535&r1=1480534&r2=1480535&view=diff
>>>
>> ==============================================================================
>>> ---
>>>
>> tomee/tomee/trunk/utils/openejb-provisionning/src/main/java/org/apache/openejb/resolver/maven/Parser.java
>>> (original)
>>> +++
>>>
>> tomee/tomee/trunk/utils/openejb-provisionning/src/main/java/org/apache/openejb/resolver/maven/Parser.java
>>> Thu May 9 06:30:22 2013
>>> @@ -19,6 +19,22 @@ package org.apache.openejb.resolver.mave
>>> import java.net.MalformedURLException;
>>> import org.ops4j.pax.url.maven.commons.MavenRepositoryURL;
>>>
>>> +/**
>>> + * This class doesn't seem compliant with the documented Maven
>> Coordinates
>>> of
>>> + *
>>> + * - groupId:artifactId:packaging:classifier:version
>>> + *
>>> + * http://maven.apache.org/pom.html#Maven_Coordinates
>>> + *
>>> + * For example, per the output of `mvn dependency:tree` TomEE is:
>>> + *
>>> + * - org.apache.openejb:apache-tomee:zip:webprofile:1.6.0-SNAPSHOT
>>> + *
>>> + * It instead seems to use its own non-maven coordinates of
>>> + *
>>> + * - groupId:artifactId:version:packaging:classifier
>>> + *
>>> + */
>>> public class Parser {
>>> public static final String VERSION_LATEST = "LATEST";
>>> private static final String SYNTAX =
>>> "mvn:[repository_url!]groupId/artifactId[/[version]/[type]]";
>>
>>
Re: Coordinates in Provisioning (was Re: svn commit: r1480535)
Posted by Romain Manni-Bucau <rm...@gmail.com>.
-1 for karaf,
-1 for aether
+0 for aether-mvn
that said that's too late David to change it (either order of info or the
name), we already released it several times
*Romain Manni-Bucau*
*Twitter: @rmannibucau <https://twitter.com/rmannibucau>*
*Blog: **http://rmannibucau.wordpress.com/*<http://rmannibucau.wordpress.com/>
*LinkedIn: **http://fr.linkedin.com/in/rmannibucau*
*Github: https://github.com/rmannibucau*
2013/5/9 David Blevins <da...@gmail.com>
> Seems like there's a simple solution, let's not use 'mvn:' as the prefix
> if we aren't using the Maven coordinate system.
>
> Any ideas on a better prefix? 'karaf:' or 'aether:'
>
> As well, do you have any reference to Aether that shows this alternate
> coordinates?
>
>
> -David
>
> On May 9, 2013, at 12:38 AM, Romain Manni-Bucau <rm...@gmail.com>
> wrote:
>
> > Hi
> >
> > That's normal:
> > - matches better default case
> > - consistent with karaf/aether
> > ---------- Message transféré ----------
> > De : <db...@apache.org>
> > Date : 9 mai 2013 08:30
> > Objet : svn commit: r1480535 -
> >
> /tomee/tomee/trunk/utils/openejb-provisionning/src/main/java/org/apache/openejb/resolver/maven/Parser.java
> > À : <co...@tomee.apache.org>
> >
> > Author: dblevins
> > Date: Thu May 9 06:30:22 2013
> > New Revision: 1480535
> >
> > URL: http://svn.apache.org/r1480535
> > Log:
> > Doesn't seem like the coordinates used in this chunk of code match the
> > maven coordinate system.
> >
> > Modified:
> >
> >
> tomee/tomee/trunk/utils/openejb-provisionning/src/main/java/org/apache/openejb/resolver/maven/Parser.java
> >
> > Modified:
> >
> tomee/tomee/trunk/utils/openejb-provisionning/src/main/java/org/apache/openejb/resolver/maven/Parser.java
> > URL:
> >
> http://svn.apache.org/viewvc/tomee/tomee/trunk/utils/openejb-provisionning/src/main/java/org/apache/openejb/resolver/maven/Parser.java?rev=1480535&r1=1480534&r2=1480535&view=diff
> >
> ==============================================================================
> > ---
> >
> tomee/tomee/trunk/utils/openejb-provisionning/src/main/java/org/apache/openejb/resolver/maven/Parser.java
> > (original)
> > +++
> >
> tomee/tomee/trunk/utils/openejb-provisionning/src/main/java/org/apache/openejb/resolver/maven/Parser.java
> > Thu May 9 06:30:22 2013
> > @@ -19,6 +19,22 @@ package org.apache.openejb.resolver.mave
> > import java.net.MalformedURLException;
> > import org.ops4j.pax.url.maven.commons.MavenRepositoryURL;
> >
> > +/**
> > + * This class doesn't seem compliant with the documented Maven
> Coordinates
> > of
> > + *
> > + * - groupId:artifactId:packaging:classifier:version
> > + *
> > + * http://maven.apache.org/pom.html#Maven_Coordinates
> > + *
> > + * For example, per the output of `mvn dependency:tree` TomEE is:
> > + *
> > + * - org.apache.openejb:apache-tomee:zip:webprofile:1.6.0-SNAPSHOT
> > + *
> > + * It instead seems to use its own non-maven coordinates of
> > + *
> > + * - groupId:artifactId:version:packaging:classifier
> > + *
> > + */
> > public class Parser {
> > public static final String VERSION_LATEST = "LATEST";
> > private static final String SYNTAX =
> > "mvn:[repository_url!]groupId/artifactId[/[version]/[type]]";
>
>