[ 
http://jira.codehaus.org/browse/SUREFIRE-700?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=259517#action_259517
 ] 

Kristian Rosenvold commented on SUREFIRE-700:
---------------------------------------------

And, as can be seen from /src/site/apt/developingSurefire.apt this solution is 
shade based and not classloader based, giving 100% isolation.

> Surefire is not isolated from itself
> ------------------------------------
>
>                 Key: SUREFIRE-700
>                 URL: http://jira.codehaus.org/browse/SUREFIRE-700
>             Project: Maven Surefire
>          Issue Type: Improvement
>          Components: classloading
>    Affects Versions: 2.4.1, 2.4.2, 2.4.3, 2.5, 2.6, 2.7, 2.7.1, 2.7.2
>            Reporter: Kristian Rosenvold
>            Assignee: Kristian Rosenvold
>             Fix For: 2.8
>
>         Attachments: stage2.patch
>
>
> The current classloader structure in surefire does not isolate surefire from 
> changes to surefire in itself. This means an interface change in *most* 
> private interfaces and classes can break the build of surefire itself.
> This is due to the classloader structure 
> systemclassloader<-testclassloader<-providerclassloader, where a modified 
> surefire immediately becomes effective by being loaded in testclassloader.
> This issue will be fixed by making the following structure:
> systemclassloader<-testframeworkclassloader<-testclassloader
> systemclassloader<-testframeworkclassloader<-surefireclassloader
> Pardon the ascii graphics but it seems like jira does not allow me to draw 
> systemclassloader<-testframeworkclassloader as one common root ;)

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to