[ 
https://issues.apache.org/jira/browse/CAMEL-9300?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14994114#comment-14994114
 ] 

ASF GitHub Bot commented on CAMEL-9300:
---------------------------------------

GitHub user astefanutti opened a pull request:

    https://github.com/apache/camel/pull/671

    CAMEL-9300: DeltaSpike container control dependency in Camel CDI shou…

    …ld be optional

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/astefanutti/camel CAMEL-9300

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/camel/pull/671.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #671
    
----
commit fb660119367c0fbde1f61e2864cd079c2037b05d
Author: Antonin Stefanutti <[email protected]>
Date:   2015-11-06T17:40:56Z

    CAMEL-9300: DeltaSpike container control dependency in Camel CDI should be 
optional

----


> DeltaSpike container control dependency in Camel CDI should be optional
> -----------------------------------------------------------------------
>
>                 Key: CAMEL-9300
>                 URL: https://issues.apache.org/jira/browse/CAMEL-9300
>             Project: Camel
>          Issue Type: Task
>          Components: camel-cdi
>    Affects Versions: 2.16.0
>            Reporter: Antonin Stefanutti
>            Priority: Minor
>
> DeltaSpike container control API is only used to bootstrap Camel CDI in Java 
> SE either directly with the {{Main}} class or via the Camel Maven plugin. 
> Hence that dependency should be declared {{optional}} so that dependent 
> projects that uses Camel CDI in non SE environments do not have to deal with 
> it. This is the case of WildFly Camel for example. Note that the DeltaSpike 
> container control packages are already excluded form OSGi imports.
> As the Camel Maven plugin does not check for optionality when adding CDI 
> extra plugin dependencies (when {{useCDI}} is set to {{true}}), it is safe to 
> declare that dependency optional.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to