Hi,

1) in 3.0.x it is afaict. In master there are more features depending on
it (investigating which ones right now).

2) Well, I'm not sure. After all this dependency is optional for Java EE
environments. Let's see what other features need this.

kind regards,
Christoph

On 28/10/14 10:57, Achim Nierbeck wrote:
> Hi,
> 
> got two questions regarding this:
> 
> - Are you sure this is the only place where this bundle is needed?
> - wouldn't it be a better fit to place this dependency to pax-cdi-weld?
> 
> regards, Achim
> 
> 
> 2014-10-28 10:46 GMT+01:00 <[email protected]>:
> 
>> Repository: karaf
>> Updated Branches:
>>   refs/heads/karaf-3.0.x 20764a594 -> ee3d7e982
>>
>>
>> [KARAF-3329] provide javax.annotation-1.2 via extra bundle
>>
>>
>> Project: http://git-wip-us.apache.org/repos/asf/karaf/repo
>> Commit: http://git-wip-us.apache.org/repos/asf/karaf/commit/ee3d7e98
>> Tree: http://git-wip-us.apache.org/repos/asf/karaf/tree/ee3d7e98
>> Diff: http://git-wip-us.apache.org/repos/asf/karaf/diff/ee3d7e98
>>
>> Branch: refs/heads/karaf-3.0.x
>> Commit: ee3d7e982200d846de68e5f35807cf398533af13
>> Parents: 20764a5
>> Author: Christoph Gritschenberger <[email protected]>
>> Authored: Tue Oct 28 10:45:48 2014 +0100
>> Committer: Christoph Gritschenberger <[email protected]>
>> Committed: Tue Oct 28 10:45:48 2014 +0100
>>
>> ----------------------------------------------------------------------
>>  assemblies/features/enterprise/src/main/feature/feature.xml | 1 +
>>  1 file changed, 1 insertion(+)
>> ----------------------------------------------------------------------
>>
>>
>>
>> http://git-wip-us.apache.org/repos/asf/karaf/blob/ee3d7e98/assemblies/features/enterprise/src/main/feature/feature.xml
>> ----------------------------------------------------------------------
>> diff --git a/assemblies/features/enterprise/src/main/feature/feature.xml
>> b/assemblies/features/enterprise/src/main/feature/feature.xml
>> index 6d23cc4..c76515e 100644
>> --- a/assemblies/features/enterprise/src/main/feature/feature.xml
>> +++ b/assemblies/features/enterprise/src/main/feature/feature.xml
>> @@ -253,6 +253,7 @@
>>      <feature name="weld" description="JBoss Weld CDI container support"
>> version="${weld.version}" resolver="(obr)">
>>          <details>Add support of JBoss Weld CDI container.</details>
>>          <feature>pax-cdi-weld</feature>
>> +        <bundle>mvn:javax.annotation/javax.annotation-api/1.2</bundle>
>>
>>  <bundle>mvn:org.jboss.weld/weld-osgi-bundle/${weld.version}</bundle>
>>      </feature>
>>
>>
>>
> 
> 


Attachment: smime.p7s
Description: S/MIME Cryptographic Signature

Reply via email to