cstamas commented on pull request #548: URL: https://github.com/apache/maven/pull/548#issuecomment-925591425
Think we have 2 directions to go: * modify/extend/customize Sisu in a way it suits us (already happened partially, think `@SessionScoped` for example) * introduce something like "maven container" (backed by Sisu) Definitely we need something, as Sisu has no notion of "class realms" (while Maven does create pretty complex class-loader hierarchy) as this issue shows. But, also as this issue shows, this was the LAST Plexus annotated component in Maven Core, and the ONLY spot where code required this "visibility filtering" thing (at least as I know of, as all the rest is plain JSR330 `@Inject`). On the other hand, people are like "maven uses Sisu, okay, read sisu doc, implement something, and bummer, this does not work" (see issue https://issues.apache.org/jira/browse/MNG-6739). This is due HOW Maven uses Sisu and again, the class loading, but this time problem is not exporting some stuff (like Guice) to plugins class path.... IMHO, this is a good topic for upcoming Aperitif... -- 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]
