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

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

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

   Yes think extensions are in scope there and don't think scope is the biggest 
code (maybe guice made it verbose but the CDI way is actually quite light - 
`Context` usage in subclass proxies which can be done at build time, even if it 
is just dumping asm one at build time and not using an annot proc or sthg more 
build oriented). But if we want to split we can maybe start by param injection 
only.




> 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