cstamas commented on pull request #88:
URL: https://github.com/apache/maven-resolver/pull/88#issuecomment-864408338


   Just to clear up things: after this PR get merged, guice as dependency (for 
guice users) and sisu as dependency (for sisu users, but it implicitly needs 
guice as well) is needed, as there is no more "poor mans DI" (service locator) 
- DI-less mode. Simply put, you need either guice or sisu (hence guice as well) 
at runtime with resolver. There is STILL possibility to go "third way", and 
manually wire up things, which I'd not recommend, but in that case you just 
need to exclude guice and sisu and you are done.


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

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to