Martin, Thanks for your effort, I will take a peek at this soon.
Just FTR, this is GH link: https://github.com/apache/maven/compare/master...Geomatys:maven:explicit-module-path Thanks T On Mon, Jan 8, 2024 at 4:10 AM Martin Desruisseaux < martin.desruisse...@geomatys.com> wrote: > Hello all > > I replaced again the commits for the JPMS proposal, with a new version > for review. > > The first commit [1] just adds @Override annotations and reduce code > redundancy in MavenProject (side-effect of the first proposal). > > The second commit [2] refactors DependencyProperties for storing values > or arbitrary types instead of Strings. The API is type-safe, with the > type of objects enforced by parameterized keys. This change is for > preparing the use of PathType values instead of boolean flags. This is > in response to last week discussion, for avoiding to restrict Maven core > to paths identified by the predefined set of flags. > > The third commit [3] adds PathType and > DependencyResolverResult.getDispatchedPath(), but without implementing > the later yet. In this new version, PathType is language-neutral and the > constants for the Java language (class-path, module-path, etc.) are > defined in the JavaPathType subclass. > > The fourth commit [4] adds the PATH_TYPES property key in > DependencyProperties, and deprecates FLAG_CLASS_PATH_CONSTITUENT and > isAddedToClasspath(). > > The fifth commit [5] implements > DependencyResolverResult.getDispatchedPaths() (proposals for better > method names are welcome). > > The sixth commit [6] addq or modifies artifact types. This commit is > likely to be incorrect after December 14th changes. > > Martin > > [1] > https://github.com/Geomatys/maven/commit/8658ba1af542f816522ea643bfd0f23200b63a78 > [2] > https://github.com/Geomatys/maven/commit/3e6e78efd6b93847868f3ee3d0fe56a6a1abb867 > [3] > https://github.com/Geomatys/maven/commit/309a4cb536aa2b799ee946f7819d90eea92adf7e > [4] > https://github.com/Geomatys/maven/commit/76e9b1161f739d21affc596095214332a87a3657 > [5] > https://github.com/Geomatys/maven/commit/a68caed0fe633a1c0f94e8840004575abbc41cf5 > [6] > https://github.com/Geomatys/maven/commit/0c4da126fd7007a6cbcf97d20434f782efb3cdc9 >
