apache / apache/maven

[MNG-8227] Cleanup after Sisu upgrade

Open
#10,783 3 comments 0 reactions 0 assignees View on GitHub
enhancement priority:major
Dominant language
Java
Stars
5.3k
Forks
3.1k
Avg merge
20h 42m
Merged PRs (30d)
297

Description

**[Konrad Windszus](https://issues.apache.org/jira/secure/ViewProfile.jspa?name=kwin)** opened **[MNG-8227](https://issues.apache.org/jira/browse/MNG-8227?redirect=false)** and commented

After the upgrade to Sisu 0.9.0 in MNG-7913 some code is no longer necessary:

[PathConverter](https://github.com/apache/maven/blob/acec540547bca289b0579ab0c8f371004b8903f1/maven-core/src/main/java/org/apache/maven/configuration/internal/DefaultBeanConfigurator.java#L175) (shipped with https://github.com/eclipse-sisu/sisu.plexus/commit/8800f2d47ac15e4e059a9dc3bd59e6a39abad49b)

---

**Issue Links:**
- [MNG-7913](https://issues.apache.org/jira/browse/MNG-7913) Upgrade Sisu version to 0.9.0.M2
(_**"is caused by"**_)

Contributor guide

Open the contributing guide

Research direction

Start with maven-core/src/main/java/org/apache/maven/configuration/internal/DefaultBeanConfigurator.java at the PathConverter reference, then read MNG-7913 to understand the Sisu 0.9.0 upgrade. Confirm which code became unnecessary after that upgrade and verify the Maven core build after cleanup.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
build-system
Issue type
Refactor
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
48/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.