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

ASF GitHub Bot commented on KARAF-6418:
---------------------------------------

nantunes commented on pull request #937: [KARAF-6418] Flag to mark transitive 
dependencies with dependency="true"
URL: https://github.com/apache/karaf/pull/937
 
 
   See https://issues.apache.org/jira/browse/KARAF-6418.
   
   Adds a flag `markTransitiveAsDependency` indicating whether the plugin 
should mark transitive dependencies' bundle elements as a dependency. Defaults 
to `false` to keep the current behaviour.
   
   For consistency, also adds the `markRuntimeScopeAsDependency` flag, 
indicating whether the plugin should also mark dependencies' with the runtime 
scope. Defaults to `true`.
   
   Note: In fact I'm needing this in the 4.2.x branch.
 
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


> Add flag to mark transitive dependencies with dependency="true"
> ---------------------------------------------------------------
>
>                 Key: KARAF-6418
>                 URL: https://issues.apache.org/jira/browse/KARAF-6418
>             Project: Karaf
>          Issue Type: New Feature
>          Components: karaf
>    Affects Versions: 4.2.6
>            Reporter: Nelson Antunes
>            Priority: Major
>
> The karaf-maven-plugin doesn't allow to specify whether bundle is dependency 
> or not solely via pom dependencies. Maintaining a manual feature file with 
> all the current dependencies seems troublesome.
> Adding a flag {{markTransitiveAsDependency}} indicating whether the plugin 
> should mark transitive dependencies' bundle elements as a dependency would 
> solve the problem. It should defaults to {{false}} to keep the current 
> behaviour and also only have an effect when {{includeTransitiveDependency}} 
> is {{true}}.
> For consistency we should also add a {{markRuntimeScopeAsDependency}} flag, 
> similarly indicating whether the plugin should also mark dependencies' with 
> the _runtime_ scope, despite the fact that behaviour doesn't seem to be 
> working ([#KARAF-4537|https://issues.apache.org/jira/browse/KARAF-4537]).
>  
> ----
> {color:#505f79}Side notes:{color}
> {color:#505f79}Isn't the {{DependencyHelper31}} effectively filtering-out 
> dependencies with the scope "provided", rendering the {{ignoreScopeProvided}} 
> flag useless? 
> ([Dependency31Helper.java#L231|https://github.com/apache/karaf/blob/master/tooling/karaf-maven-plugin/src/main/java/org/apache/karaf/tooling/utils/Dependency31Helper.java#L231]){color}
> {color:#505f79}Also, isn't it making the {{simplifyBundleDependencies}} flag 
> useless, by always removing feature transitive dependencies regardless? 
> ([Dependency31Helper.java#L225|https://github.com/apache/karaf/blob/master/tooling/karaf-maven-plugin/src/main/java/org/apache/karaf/tooling/utils/Dependency31Helper.java#L225]){color}



--
This message was sent by Atlassian Jira
(v8.3.2#803003)

Reply via email to