gnodet commented on PR #232:
URL: 
https://github.com/apache/maven-integration-testing/pull/232#issuecomment-1382840257

   > 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 
https://github.com/apache/maven-integration-testing/pull/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.
   
   
   


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