-
Notifications
You must be signed in to change notification settings - Fork 125
Use Maven 4 as the runtime for m2eclipse #2053
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
Test Results 330 files +3 330 suites +3 58m 44s ⏱️ - 18m 58s For more details on these failures and errors, see this check. Results for commit d2927b5. ± Comparison against base commit d18979f. ♻️ This comment has been updated with latest results. |
cda130d to
017a8bf
Compare
a954c36 to
a953a87
Compare
|
FYI @stbischof and @danthe1st as I know you are interested in Maven 4 support. This at least already compiles ... |
|
Whooop whooop!!!!! Thank you a lot for your work |
|
All tests are passing... except some of the flapping ones. So that's already promising, and as there is discussions about release Maven 4 soon (finally, really!) I think we then can quite easily upgrade here. So next logical step would be to create a Maven 4 testcase to see if maven features actually work! |
|
We get a lot of deprecation warnings now but that is expected and we should fix those later on. One thing we need to check is if the new runtime is actually used for run the tests! |
a953a87 to
dd28e85
Compare
6bbf4e7 to
430816a
Compare
|
It seems I was a bit to optimistic here, while the runtime itself compiles, due to we include the old ones as well these where picked up, if I enforce Maven 4 runtime the test fails (see previous commit) and if I enforce it to be used for compilation there are also compile error... so we need to check if
|
|
Wouldn't apache/maven#2380 supposed to help in this case? |
1d41df7 to
a177c2b
Compare
|
See also discussion here: |
f6c3302 to
6b69674
Compare
|
@HannesWell this is currently only a draft so may contain some changes that must be cleaned up before a final version. I now have made it to compile again with Maven4 but possibly retain backwards compatible with Maven 3.9.x ... |
|
Maven has added some backward compatibility support now so with next release some of the things might no longer be needed, still we have some issues where we extend maven classes that now uses JSR330 in maven 4 with constructor injections. I need to check how we can overcome this. |
8aabf8e to
cf1fed4
Compare
|
Updated to Maven4-rc5 (currently 545 failing tests) |
be126ca to
e0b4bbd
Compare
ad5a7b0 to
2ff3882
Compare
|
With #2105 we are now down to 345 failing test... a lot seem about failing project configuration, some where we try to mutate now immutable objects. |
18b8071 to
e5d18fc
Compare
|
I added a null check and a catch for the unsupported operation, both seem to indicate some change in Maven 4 that we need to consider here. This should let some more test getting further (not that effective now 343 failing test...). |
e5d18fc to
e4b432e
Compare
|
We are currently hit by this bug: (now 303 failing tests...) |
Maven 4 will soon be released and should be able to execute Maven 3 projects as well. We should therefore upgrade m2e to use Maven 4 from now on. For users that require Maven 3.9 line we can offer the latest 3.9.11 maven runtime as an additional install option like we already did for Maven 3.9.6
e4b432e to
d2927b5
Compare
Maven 4 will soon be released and should be able to execute Maven 3 projects as well. We should therefore upgrade m2e to use Maven 4 from now on.
For users that require Maven 3.9 line we can offer the latest 3.9.11 maven runtime as an additional install option like we already did for Maven 3.9.6