Skip to content

Commit 3666eb1

Browse files
committed
Review updates
1 parent ca6a503 commit 3666eb1

5 files changed

Lines changed: 16 additions & 29 deletions

File tree

.github/workflows/nightly.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -922,7 +922,7 @@ jobs:
922922
mvn archetype:generate -DinteractiveMode=false -DaskForDefaultPropertyValues=false -DarchetypeGroupId=org.eclipse.starter -DarchetypeArtifactId=jakarta-starter -DarchetypeVersion=2.8.0-SNAPSHOT -DjakartaVersion=11 -Dprofile=full -DjavaVersion=17 -Druntime=wildfly -DoutputDirectory=app/wildfly -Dgoals="clean package" | tee mvn_output.txt
923923
924924
MAVEN_EXIT_CODE=${PIPESTATUS[0]}
925-
ERROR_MESSAGE="only Payara currently supports Jakarta EE 11 in a stable release"
925+
ERROR_MESSAGE="WildFly does not support Jakarta EE 11"
926926
if ! { [ $MAVEN_EXIT_CODE -ne 0 ] && grep -q "$ERROR_MESSAGE" mvn_output.txt; }; then
927927
echo "Maven build did not fail, or the expected error message was not found. Test failed."
928928
exit 1

archetype/src/main/resources/META-INF/archetype-post-generate.groovy

Lines changed: 10 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -46,13 +46,6 @@ private validateInput(jakartaVersion, profile, javaVersion, runtime, docker, Fil
4646
throw new RuntimeException("Failed, valid Docker options are yes and no")
4747
}
4848

49-
// For now, this catch-all will simplify the rest of the validation.
50-
// It should be evolved once more runtimes properly suppport Jakarta EE 11.
51-
if ((jakartaVersion == '11') && (runtime != 'none') && (runtime != 'payara')) {
52-
FileUtils.forceDelete(outputDirectory)
53-
throw new RuntimeException("Failed, only Payara currently supports Jakarta EE 11 in a stable release")
54-
}
55-
5649
if ((profile == 'core') && (Double.valueOf(jakartaVersion) < 10)) {
5750
FileUtils.forceDelete(outputDirectory)
5851
throw new RuntimeException("Failed, the Core Profile is only supported for Jakarta EE 10 and above")
@@ -69,6 +62,11 @@ private validateInput(jakartaVersion, profile, javaVersion, runtime, docker, Fil
6962
}
7063

7164
if (runtime == 'glassfish') {
65+
if ((jakartaVersion == '11')) {
66+
FileUtils.forceDelete(outputDirectory)
67+
throw new RuntimeException("Failed, GlassFish does not support Jakarta EE 11")
68+
}
69+
7270
if (profile == 'core') {
7371
FileUtils.forceDelete(outputDirectory)
7472
throw new RuntimeException("Failed, GlassFish does not support the Core Profile")
@@ -127,6 +125,11 @@ private validateInput(jakartaVersion, profile, javaVersion, runtime, docker, Fil
127125
}
128126

129127
if (runtime == 'wildfly') {
128+
if ((jakartaVersion == '11')) {
129+
FileUtils.forceDelete(outputDirectory)
130+
throw new RuntimeException("Failed, WildFly does not support Jakarta EE 11")
131+
}
132+
130133
if ((jakartaVersion == '9') || (jakartaVersion == '9.1')) {
131134
FileUtils.forceDelete(outputDirectory)
132135
throw new RuntimeException("Failed, WildFly does not offer a stable release for Jakarta EE 9 or Jakarta EE 9.1")

archetype/src/main/resources/archetype-resources/src/main/liberty/config/server.xml

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -18,18 +18,12 @@
1818
#else
1919
<feature>webProfile-10.0</feature>
2020
#end
21-
#elseif (${jakartaVersion} == '11')
21+
#else
2222
#if (${profile} == 'full')
2323
<feature>jakartaee-11.0</feature>
2424
#else
2525
<feature>webProfile-11.0</feature>
2626
#end
27-
#else
28-
#if (${profile} == 'full')
29-
<feature>jakartaee-12.0</feature>
30-
#else
31-
<feature>webProfile-12.0</feature>
32-
#end
3327
#end
3428
</featureManager>
3529
<basicRegistry id="defaultBasicRegistry" realm="BasicRealm">

ui/src/main/java/org/eclipse/starter/ui/Project.java

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -277,11 +277,7 @@ private void updateRuntimeEnabledStates() {
277277
runtimes.get("glassfish").setDisabled(false);
278278
}
279279

280-
if (jakartaVersion == 12) {
281-
runtimes.get("open-liberty").setDisabled(true);
282-
} else {
283-
runtimes.get("open-liberty").setDisabled(false);
284-
}
280+
runtimes.get("open-liberty").setDisabled(false);
285281

286282
if (jakartaVersion == 8 && javaVersion == 21) {
287283
// Payara 5 does not support Java SE 21.

ui/src/main/webapp/index.xhtml

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -31,10 +31,7 @@
3131
</p:selectOneMenu>
3232
<h:outputText
3333
styleClass="footnote"
34-
value="Jakarta EE 11 requires Java SE 17 or later."/>
35-
<h:outputText
36-
styleClass="footnote"
37-
value="Jakarta EE 10 requires Java SE 11 or later."/>
34+
value="Jakarta EE 11 requires Java SE 17 or later, Jakarta EE 10 requires Java SE 11 or later."/>
3835
</p:column>
3936

4037
<p:column>
@@ -55,7 +52,7 @@
5552
</p:selectOneMenu>
5653
<h:outputText
5754
styleClass="footnote"
58-
value="Core Profile only available for Jakarta EE 10 and later."/>
55+
value="Core Profile is only available for Jakarta EE 10 and later."/>
5956
</p:column>
6057

6158
<p:column>
@@ -76,10 +73,7 @@
7673
</p:selectOneMenu>
7774
<h:outputText
7875
styleClass="footnote"
79-
value="Java SE 11 requires Jakarta EE below 11."/>
80-
<h:outputText
81-
styleClass="footnote"
82-
value="Java SE 8 requires Jakarta EE below 10."/>
76+
value="Java SE 11 requires Jakarta EE 10 or earlier, Java SE 8 requires Jakarta EE 9.1 or earlier."/>
8377
</p:column>
8478

8579
<p:column>

0 commit comments

Comments
 (0)