karaf-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jean-Baptiste Onofré ...@nanthrax.net>
Subject Re: svn commit: r1346392 - /karaf/trunk/assemblies/features/framework/src/main/resources/resources/etc/org.ops4j.pax.url.mvn.cfg
Date Wed, 06 Jun 2012 08:22:42 GMT
Just to explain, pax-url* should be in the system repo, pax-web* not as 
an optional issue.

IMHO, for pax-web, we should not add SNAPSHOT (artifacts or repo), but 
more wait for a release (that we can do easily and quickly).

Regards
JB


On 06/06/2012 10:19 AM, Jean-Baptiste Onofré wrote:
> Correct me if I'm wrong but those artifacts should be in the system
> repo. So, it's not required to have the snapshot repository in the
> org.ops4j.pax.url.mvn.cfg file as it should be resolved from system repo.
>
> We have two cases:
> 1/ the artifacts are not in the system repo => we have an assembly issue
> 2/ the artifacts are in the system repo but not used => we have an issue
> around pax-url-aether (config for instance)
>
> My 0.02€
>
> Regards
> JB
>
> On 06/05/2012 10:30 PM, Andreas Pieber wrote:
>> Tbh I do not like this commit. Nevertheless it shows that this is really
>> the problem. Moving the entry directly into the test setup and reverting
>> this commit would have the same Effekt but would be rather non intrusive.
>> At least we can't forget to remove it from the final distribution again
>>
>> Kind regards Andreas
>> On Jun 5, 2012 4:08 PM,<cschneider@apache.org> wrote:
>>
>>> Author: cschneider
>>> Date: Tue Jun 5 14:07:42 2012
>>> New Revision: 1346392
>>>
>>> URL: http://svn.apache.org/viewvc?rev=1346392&view=rev
>>> Log:
>>> KARAF-1522 Add pax snapshot repo to karaf distro
>>>
>>> Modified:
>>>
>>> karaf/trunk/assemblies/features/framework/src/main/resources/resources/etc/org.ops4j.pax.url.mvn.cfg
>>>
>>>
>>> Modified:
>>> karaf/trunk/assemblies/features/framework/src/main/resources/resources/etc/org.ops4j.pax.url.mvn.cfg
>>>
>>> URL:
>>> http://svn.apache.org/viewvc/karaf/trunk/assemblies/features/framework/src/main/resources/resources/etc/org.ops4j.pax.url.mvn.cfg?rev=1346392&r1=1346391&r2=1346392&view=diff
>>>
>>>
>>> ==============================================================================
>>>
>>> ---
>>> karaf/trunk/assemblies/features/framework/src/main/resources/resources/etc/org.ops4j.pax.url.mvn.cfg
>>>
>>> (original)
>>> +++
>>> karaf/trunk/assemblies/features/framework/src/main/resources/resources/etc/org.ops4j.pax.url.mvn.cfg
>>>
>>> Tue Jun 5 14:07:42 2012
>>> @@ -95,5 +95,5 @@ org.ops4j.pax.url.mvn.defaultLocalRepoAs
>>> org.ops4j.pax.url.mvn.repositories= \
>>> http://repo1.maven.org/maven2@id=central, \
>>>
>>> http://repository.apache.org/content/groups/snapshots-group@id=apache@snapshots@noreleases,
>>>
>>> \
>>> +
>>> https://oss.sonatype.org/content/repositories/ops4j-snapshots@id=ops4j.sonatype.snapshots.deploy@snapshots@noreleases,
>>>
>>> \
>>> file:${karaf.home}/${karaf.default.repository}@id=systemrepo
>>> -
>>>
>>>
>>>
>>
>

-- 
Jean-Baptiste Onofré
jbonofre@apache.org
http://blog.nanthrax.net
Talend - http://www.talend.com

Mime
View raw message