[ 
https://jira.codehaus.org/browse/MNG-1945?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=338839#comment-338839
 ] 

Vincent Massol commented on MNG-1945:
-------------------------------------

Hi Jason. Would have been nice to explain the reason you're closing this issue. 
In general closing an issue without an explanation is slightly disrespectful 
for whoever took the time to create the issue :) It's also interesting for 
others to know why you're doing so.

Also, I don't understand the "not a bug" reason. Of course it's not a bug; I've 
never raised this as a bug but as an Improvement.

Thanks
                
> project.getBuild().setSourceDirectory() should modify the compile source 
> roots automatically
> --------------------------------------------------------------------------------------------
>
>                 Key: MNG-1945
>                 URL: https://jira.codehaus.org/browse/MNG-1945
>             Project: Maven 2 & 3
>          Issue Type: Improvement
>          Components: POM
>    Affects Versions: 2.0.1
>            Reporter: Vincent Massol
>            Assignee: Jason van Zyl
>             Fix For: Issues to be reviewed for 3.x
>
>
> Here's the code that I have in the clover plugin right now:
>     private void redirectSourceDirectories()
>     {
>         String oldSourceDirectory = 
> this.project.getBuild().getSourceDirectory();
>         this.project.getBuild().setSourceDirectory( 
> this.cloverOutputSourceDirectory );
>         
>         // Maven2 limitation: changing the source directory doesn't change 
> the compile source roots
>         Iterator sourceRoots = 
> this.project.getCompileSourceRoots().iterator();
>         for (int i = 0; sourceRoots.hasNext(); i++)
>         {
>             String sourceRoot = (String) 
> this.project.getCompileSourceRoots().get( i );
>             if (sourceRoot.equals(oldSourceDirectory))
>             {
>                 this.project.getCompileSourceRoots().remove( i );
>                 // Note: Ideally we should add the new compile source root at 
> the same place as the
>                 // one we're removing but there's no API for this...
>                 this.project.addCompileSourceRoot( 
> this.project.getBuild().getSourceDirectory() );
>             }
>         }
>     }
> I believe this could be put in Maven core.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to