michael-o commented on PR #232:
URL: 
https://github.com/apache/maven-integration-testing/pull/232#issuecomment-1382882323

   > > I consider this wrong because they should be at most managed from within 
Verifier. Those are _system_ properties and when these are required then a 
_forked_ verifier must be used.
   > 
   > I don't really see how that's different from #232
   > 
   > Another possibility is to have the Verifier set this property explicitly 
to the value `${maven.home}/conf`. And if you think it's wrong that the 
Verifier change these properties, then let's do it in this PR as this won't 
affect anyone else.
   > 
   > But really, having the tests not being isolated is a bad idea.
   
   I am not isolation, but isolation needs to happen by the framework executing 
the tests and if necessary, create a new process for it.


-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to