From e4a560a998f2efea1356e65f06ae755c03b25e65 Mon Sep 17 00:00:00 2001
From: soloturn <soloturn@gmail.com>
Date: Thu, 10 Feb 2022 09:23:41 +0100
Subject: [PATCH] permit "mvn test" with java8.

activate javac --release only on higher java versions. it allows to compile
with java11+, but permits afterwards running tests with java8 to make sure
rendering errors can be catched.

parmit the syntax of the source to include java17, and tell the user about
it instead of just failing the compilation.
---
 pom.xml | 23 ++++++++++++++++++++++-
 1 file changed, 22 insertions(+), 1 deletion(-)

diff --git a/pom.xml b/pom.xml
index 250e37f7f..013a4cf9a 100644
--- a/pom.xml
+++ b/pom.xml
@@ -56,7 +56,6 @@
   </parent>
   <properties>
     <finalName>${project.artifactId}-${project.version}</finalName>
-    <maven.compiler.release>8</maven.compiler.release>
     <main.class>net.sourceforge.plantuml.Run</main.class>
   </properties>
   <dependencyManagement>
@@ -232,6 +231,28 @@
     </plugins>
   </build>
   <profiles>
+    <profile>
+      <!-- set the release flag only for jdk which do support it, plantuml should keep running with java8. -->
+      <id>javac-release-flag</id>
+      <activation>
+        <jdk>[9,)</jdk>
+      </activation>
+      <properties>
+        <maven.compiler.release>8</maven.compiler.release>
+      </properties>
+    </profile>
+    <profile>
+      <!-- the source contains java11+ syntax, let mvn shout. allow "mvn test" with java8 as rendering sometimes
+      can give different results with java8. -->
+      <id>javac-source-flag</id>
+      <activation>
+        <activeByDefault>true</activeByDefault>
+      </activation>
+      <properties>
+        <maven.compiler.source>17</maven.compiler.source>
+        <maven.compiler.target>1.8</maven.compiler.target>
+      </properties>
+    </profile>
     <profile>
       <id>pdf</id>
       <activation>