Re: [PR] [MNG-7266] Remove maven-compat [maven]

2024-07-03 Thread via GitHub
gnodet commented on PR #1487: URL: https://github.com/apache/maven/pull/1487#issuecomment-2205390080 > > > > > > I did investigate again and now I understand what actually happens: When I run current reporting plugins wich do _not_ depend on Maven Compat I still get CNFE, e.g., RepositorySy

Re: [PR] [MNG-7266] Remove maven-compat [maven]

2024-07-03 Thread via GitHub
gnodet commented on PR #1487: URL: https://github.com/apache/maven/pull/1487#issuecomment-2205267196 > > > > > I did investigate again and now I understand what actually happens: When I run current reporting plugins wich do _not_ depend on Maven Compat I still get CNFE, e.g., RepositorySyst

Re: [PR] [MNG-7266] Remove maven-compat [maven]

2024-07-02 Thread via GitHub
michael-o commented on PR #1487: URL: https://github.com/apache/maven/pull/1487#issuecomment-2205220610 > > > > I did investigate again and now I understand what actually happens: When I run current reporting plugins wich do _not_ depend on Maven Compat I still get CNFE, e.g., RepositorySys

Re: [PR] [MNG-7266] Remove maven-compat [maven]

2024-07-02 Thread via GitHub
gnodet commented on PR #1487: URL: https://github.com/apache/maven/pull/1487#issuecomment-2205195453 > > > > I did investigate again and now I understand what actually happens: When I run current reporting plugins wich do _not_ depend on Maven Compat I still get CNFE, e.g., RepositorySystem

Re: [PR] [MNG-7266] Remove maven-compat [maven]

2024-07-02 Thread via GitHub
gnodet commented on PR #1487: URL: https://github.com/apache/maven/pull/1487#issuecomment-2203324239 > > > > > > > These two: > > > > > > > > > > > > > > * maven-plugin-tools/maven-plugin-plugin > > > > > > > * maven-project-info-reports-plugin > > > > > > > > > > > > > >

Re: [PR] [MNG-7266] Remove maven-compat [maven]

2024-07-02 Thread via GitHub
michael-o commented on PR #1487: URL: https://github.com/apache/maven/pull/1487#issuecomment-2203132098 > > > > > > These two: > > > > > > > > > > > > * maven-plugin-tools/maven-plugin-plugin > > > > > > * maven-project-info-reports-plugin > > > > > > > > > > > > > > > ma

Re: [PR] [MNG-7266] Remove maven-compat [maven]

2024-07-02 Thread via GitHub
michael-o commented on PR #1487: URL: https://github.com/apache/maven/pull/1487#issuecomment-2203130039 > > > > > These two: > > > > > > > > > > * maven-plugin-tools/maven-plugin-plugin > > > > > * maven-project-info-reports-plugin > > > > > > > > > > > > maven-plugin-too

Re: [PR] [MNG-7266] Remove maven-compat [maven]

2024-06-14 Thread via GitHub
michael-o commented on PR #1487: URL: https://github.com/apache/maven/pull/1487#issuecomment-2168164269 Yet another finding: ``` $ for proj in maven-doxia* maven-reporting-* maven-site-plugin; do grep -r -F -f maven-compat-classes --include='*/src/main/java/**.java' $proj ; done | gr

Re: [PR] [MNG-7266] Remove maven-compat [maven]

2024-06-14 Thread via GitHub
michael-o commented on PR #1487: URL: https://github.com/apache/maven/pull/1487#issuecomment-2168030966 @Giovds please rebase -- 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.

Re: [PR] [MNG-7266] Remove maven-compat [maven]

2024-06-07 Thread via GitHub
michael-o commented on PR #1487: URL: https://github.com/apache/maven/pull/1487#issuecomment-2154676402 Preparing Maven 3.6.3 releases are ongoing. As soon as they are finished I will rebase my work on top of these and check all UTs and ITs. -- This is an automated message from the Apache

Re: [PR] [MNG-7266] Remove maven-compat [maven]

2024-05-27 Thread via GitHub
cstamas commented on PR #1487: URL: https://github.com/apache/maven/pull/1487#issuecomment-2133427438 Cruft piled up in Maven should not be underestimated :smile: -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the UR

Re: [PR] [MNG-7266] Remove maven-compat [maven]

2024-05-27 Thread via GitHub
michael-o commented on PR #1487: URL: https://github.com/apache/maven/pull/1487#issuecomment-2132856197 > > > @cstamas @michael-o one blocker after upgrading to latest site is [AbstractProjectInfoReport.java](https://github.com/apache/maven-project-info-reports-plugin/blob/master/src/main/ja

Re: [PR] [MNG-7266] Remove maven-compat [maven]

2024-05-27 Thread via GitHub
gnodet commented on PR #1487: URL: https://github.com/apache/maven/pull/1487#issuecomment-2132819017 > > @cstamas @michael-o one blocker after upgrading to latest site is [AbstractProjectInfoReport.java](https://github.com/apache/maven-project-info-reports-plugin/blob/master/src/main/java/or

Re: [PR] [MNG-7266] Remove maven-compat [maven]

2024-05-27 Thread via GitHub
michael-o commented on PR #1487: URL: https://github.com/apache/maven/pull/1487#issuecomment-2132779147 > @cstamas @michael-o one blocker after upgrading to latest site is [AbstractProjectInfoReport.java](https://github.com/apache/maven-project-info-reports-plugin/blob/master/src/main/java/o

Re: [PR] [MNG-7266] Remove maven-compat [maven]

2024-05-26 Thread via GitHub
gnodet commented on PR #1487: URL: https://github.com/apache/maven/pull/1487#issuecomment-2132769075 @cstamas @michael-o one blocker after upgrading to latest site is https://github.com/apache/maven-project-info-reports-plugin/blob/master/src/main/java/org/apache/maven/report/projectinfo/Abs

Re: [PR] [MNG-7266] Remove maven-compat [maven]

2024-04-29 Thread via GitHub
cstamas commented on PR #1487: URL: https://github.com/apache/maven/pull/1487#issuecomment-2082768935 Fix for m-p-p on the way https://github.com/apache/maven-plugin-tools/pull/280 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Git

Re: [PR] [MNG-7266] Remove maven-compat [maven]

2024-04-29 Thread via GitHub
michael-o commented on PR #1487: URL: https://github.com/apache/maven/pull/1487#issuecomment-2082487297 > > Someone needs to fix maven-plugin-plugin first: > > Does this still stands? Or latest one works? Both are unrelated. Maven Plugin Plugin remains broken without Maven Compa

Re: [PR] [MNG-7266] Remove maven-compat [maven]

2024-04-29 Thread via GitHub
cstamas commented on PR #1487: URL: https://github.com/apache/maven/pull/1487#issuecomment-2082445871 > Someone needs to fix maven-plugin-plugin first: > Does this still stands? Or latest one works? -- This is an automated message from the Apache Git Service. To respond to the m

Re: [PR] [MNG-7266] Remove maven-compat [maven]

2024-04-28 Thread via GitHub
michael-o commented on PR #1487: URL: https://github.com/apache/maven/pull/1487#issuecomment-2081432308 As far as I can see with ``` commit 7c8334ac2466b05ca0cdbeb265dc4b85b0099a2a Author: Michael Osipov Date: 2023-03-18T23:00:40+01:00 [DOXIASITETOOLS-294] Replace le

Re: [PR] [MNG-7266] Remove maven-compat [maven]

2024-04-28 Thread via GitHub
michael-o commented on PR #1487: URL: https://github.com/apache/maven/pull/1487#issuecomment-2081428694 Someone needs to fix maven-plugin-plugin first: ``` [INFO] [INFO] --- jar:3.3.0:jar (default-jar) @ maven-site-plugin --- [INFO] Building jar: /var/osipovmi/Projekte/maven-site

Re: [PR] [MNG-7266] Remove maven-compat [maven]

2024-04-27 Thread via GitHub
cstamas commented on PR #1487: URL: https://github.com/apache/maven/pull/1487#issuecomment-2080430009 For start, maven-compat is really just test dep, not needed at runtime https://github.com/apache/maven-doxia-sitetools/pull/151 -- This is an automated message from the Apache Git Servi

Re: [PR] [MNG-7266] Remove maven-compat [maven]

2024-04-26 Thread via GitHub
michael-o commented on PR #1487: URL: https://github.com/apache/maven/pull/1487#issuecomment-2079743348 > @michael-o if you look at failed CI jobs, they are due msite plugin wanting stuff that is actually removed (maven-compat). Am interested could you -- maybe even within this PR -- set up

Re: [PR] [MNG-7266] Remove maven-compat [maven]

2024-04-26 Thread via GitHub
cstamas commented on PR #1487: URL: https://github.com/apache/maven/pull/1487#issuecomment-2079652561 @michael-o if you look at failed CI jobs, they are due msite plugin wanting stuff that is actually removed (maven-compat). Am interested could you -- maybe even within this PR -- set up M v

Re: [PR] [MNG-7266] Remove maven-compat [maven]

2024-04-26 Thread via GitHub
michael-o commented on PR #1487: URL: https://github.com/apache/maven/pull/1487#issuecomment-2079429030 > @michael-o can we have a taste on new Ms from site suite? You want to me run the entire test setup of Doxia/MSITE w/o depending on maven-compat or with with PR of Maven 4? -- T

Re: [PR] [MNG-7266] Remove maven-compat [maven]

2024-04-26 Thread via GitHub
Giovds commented on PR #1487: URL: https://github.com/apache/maven/pull/1487#issuecomment-2079380222 I've noticed in the .svg that the `compat` module sits between `core` and `wagon-provider-api`. What is the next step for the `wagon-provider-api`? Does this mean we remove the `wagon-

Re: [PR] [MNG-7266] Remove maven-compat [maven]

2024-04-26 Thread via GitHub
cstamas commented on PR #1487: URL: https://github.com/apache/maven/pull/1487#issuecomment-2079366763 @michael-o can we have a taste on new Ms from site suite? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL ab

Re: [PR] [MNG-7266] Remove maven-compat [maven]

2024-04-26 Thread via GitHub
cstamas commented on PR #1487: URL: https://github.com/apache/maven/pull/1487#issuecomment-2079366084 maven-site plugin is still maven2 dependant... -- 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 t

[PR] [MNG-7266] Remove maven-compat [maven]

2024-04-26 Thread via GitHub
Giovds opened a new pull request, #1487: URL: https://github.com/apache/maven/pull/1487 Following this checklist to help us incorporate your contribution quickly and easily: - [x] Make sure there is a [JIRA issue](https://issues.apache.org/jira/browse/MNG) filed for the