-
Notifications
You must be signed in to change notification settings - Fork 843
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
Payara 7 and Jakarta EE 11 support #7716
base: master
Are you sure you want to change the base?
Conversation
going to restart CI with enterprise jobs enabled. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I only eyeballed this, but looks sane to me. I left a few inline comments, but nothing major.
@@ -30,7 +30,7 @@ | |||
<goal>compiler:compile</goal> | |||
<goal>war:exploded</goal> | |||
<goal>payara-micro:stop</goal> | |||
<goal>payara-micro:start</goal> | |||
<goal>payara-micro:dev</goal> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will this cause problems for existing installations? The dev-Mojo is only 9 months old, so old projects might hold plugin versions, not supporting this.
@@ -387,7 +392,7 @@ public boolean equals(final PayaraPlatformVersionAPI version) { | |||
return this.major == version.getMajor() | |||
&& this.minor == version.getMinor() | |||
&& this.update == version.getUpdate() | |||
&& this.build == version.getBuild(); | |||
&& (this.build == null ? version.getBuild() == null : this.build.equals(version.getBuild())); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
&& (this.build == null ? version.getBuild() == null : this.build.equals(version.getBuild())); | |
&& Objects.equals(this.build, version.getBuild())); |
assertTrue(major == version.getMajor() | ||
&& minor == version.getMinor() | ||
&& update == version.getUpdate() | ||
&& build == version.getBuild()); | ||
&& (build == null ? version.getBuild() == null : build.equals(version.getBuild()))); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
&& (build == null ? version.getBuild() == null : build.equals(version.getBuild()))); | |
&& Objects.equals(build, version.getBuild())); |
This PR introduces the following changes to support Payara 7 and Jakarta EE 11 in Apache NetBeans:
Updated Build Type Handling:
Changed the method signature from
short getBuild()
toString getBuild()
. This change allows the system to recognize and list the latestAlpha
versions of Payara 7: https://mvnrepository.com/artifact/fish.payara.distributions/payara/7.2024.1.Alpha1Alpha/Beta Version Filtering:
Only the latest
Alpha
orBeta
version will now be listed in the available versions. OtherAlpha
orBeta
versions will be skipped to maintain a cleaner and more relevant version list.Archetype Update for Payara Micro:
The archetype
fish.payara.starter:payara-starter-archetype
will now be used for the Payara Micro 6+.