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

Carsten Ziegeler commented on FELIX-4530:
-----------------------------------------

I've just committed a potential fix to the maven plugin - at least this fixes 
[[email protected]] problem (FELIX-4535).
With the change:
a) if the output directory of the plugin is not the classes directory, it is 
added to the maven resources (if not already) - this keeps the behaviour if a 
different output directory is configured
b) if the output directory is not the classes directory, it directly sets the 
Include-Resource directive for the bnd plugin and explicitely adds only the 
generated files
This should give us the best of all worlds in one setup.
I'll leave this open for some time to wait for potential feedback

> [SCR Plugin] Revisit setting the default output to target/classes
> -----------------------------------------------------------------
>
>                 Key: FELIX-4530
>                 URL: https://issues.apache.org/jira/browse/FELIX-4530
>             Project: Felix
>          Issue Type: Bug
>          Components: SCR Tooling
>    Affects Versions: maven-scr-plugin 1.17.0
>            Reporter: Felix Meschberger
>            Assignee: Carsten Ziegeler
>            Priority: Critical
>             Fix For: maven-scr-plugin 1.17.2
>
>
> With FELIX-4241 the default value for the descriptor output files has been 
> set to {{target/classes}}. Unfortunately this causes {{target/classes}} to be 
> added to the Maven Input Resources resulting in all contents of 
> {{target/classes}} to be included in the final bundle regardless of whether 
> this is actually desired or not.
> Regardless of tooling integration noted in FELIX-4241, it is against the 
> spirit of the bundle plugin to blindly include everything in 
> {{target/classes}} in the bundle. So the resolution for FELIX-4241 must be 
> revisited.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to