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

Robert Scholte commented on MNG-5899:
-------------------------------------

I think we should revise this *after* splitting up the build-pom and 
consumer-pom. I can imagine that during the construction of a buildplan the 
dependencies to a reactor module don't have to be fully resolved yet. That can 
be done after that module has been build or actually when it is required by 
another module. In the end the result must be the same as using that dependency 
from the local repsoitory.

> Reactor is no longer using the dependency reduced pom created by 
> maven-shade-plugin
> -----------------------------------------------------------------------------------
>
>                 Key: MNG-5899
>                 URL: https://issues.apache.org/jira/browse/MNG-5899
>             Project: Maven
>          Issue Type: Bug
>    Affects Versions: 3.3.1, 3.3.3
>            Reporter: Trask Stalnaker
>
> This works with maven 3.2.5 and prior, but does not work with the latest 
> releases (3.3.1 and 3.3.3), or master (tested against 3.3.7-SNAPSHOT).
> I did some bisecting and found the commit that caused the breakage:
> https://github.com/apache/maven/commit/be3fb200326208ca4b8c41ebf16d5ae6b8049792
> In particular, this commit introduced some code to not rebuild the model if 
> it was already loaded during reactor resolution.
> This is causing maven not to use the dependency reduced pom created by the 
> maven-shade-plugin.
> Removing this code fixes the situation.  I'm hoping that code was just 
> introduced as a performance optimization, and that I haven't broken anything 
> functionally by removing it (the tests still pass at least).
> I will submit a pull request with the change for review.
> Here is the original maven-shade-plugin issue as reported by the Apache Storm 
> team: https://issues.apache.org/jira/browse/MSHADE-206



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to