Details
-
Task
-
Status: Closed
-
Major
-
Resolution: Fixed
-
None
-
None
Description
maven-resolver currently supports:
- ServiceLocator (deprecated as of 1.7.0 see
MRESOLVER-160) - "vanilla" Guice (provides a module)
- DI using Sisu, as used in Maven
IMO, it makes not much sense to support 3 vastly different "DI"s (in quotes as ServiceLocator is really just a dumb factory pattern).
Not only just complicates the code base, makes changes error prone at least, yields for "exceptions" (this or that will never work with it, as seen with SyncContext), and, for me most importantly, prevents proper constructor initialization and validation of components.
Attachments
Issue Links
- relates to
-
MRESOLVER-387 Provide "static" supplier for RepositorySystem
- Closed
-
MRESOLVER-169 Drop use of ServiceLocator
- Closed
-
MRESOLVER-386 Make all injected ctors public, deprecate all def ctors
- Closed
-
MRESOLVER-411 Update to Maven 4.0.0-alpha-8
- Closed
- links to