[ 
https://issues.apache.org/jira/browse/MNG-7954?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17811377#comment-17811377
 ] 

ASF GitHub Bot commented on MNG-7954:
-------------------------------------

gnodet commented on PR #1393:
URL: https://github.com/apache/maven/pull/1393#issuecomment-1912563154

   > Think we should start minimalistic, maven IoC need is actually pretty low 
- mainly bean definition + overriding - so less we keep better it is in terms 
of maintenance. Rewriting from scratch gives the opportunity to drop guice and 
sisu at some point too so can be quite promishing if we manage to make 
plexus-world optional (realm = TCCL) but this is likely mvn 6 or 7 ;).
   
   Yeah, I think I'll even drop support for scopes for now, we don't really 
need it for plugins.  Those are only needed when dealing with extensions or 
maven-core, as plugins are always bound to the mojo execution.




> Provide a cleaner DI api
> ------------------------
>
>                 Key: MNG-7954
>                 URL: https://issues.apache.org/jira/browse/MNG-7954
>             Project: Maven
>          Issue Type: New Feature
>          Components: API
>            Reporter: Guillaume Nodet
>            Priority: Major
>             Fix For: 4.0.0
>
>
> With https://issues.apache.org/jira/browse/MNG-7947  the {{jakarta.inject}} 
> package has been brought into the API.
> We need a cleaner way and not depend on any third party library if possible.
> The 
> [{{SessionScope}}|https://github.com/apache/maven/blob/23bca281fcd084ac21d80f5a2950dcee30a19080/maven-core/src/main/java/org/apache/maven/session/scope/internal/SessionScope.java#L123-L125]
>  would also require some cleaning to avoid having to rely on 
> {{{}(org.eclipse.sisu|javax.enterprise.inject|jakarta.enterprise.inject).Typed{}}}.
> For complete DI, we may also miss the sisu annotations 
> ({{{}PostConstruct{}}}, {{PreDestroy}} and {{Priority}} and 
> {{{}EagerSingleton{}}}.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to