[MSHARED-826] Require Java 7#7
[MSHARED-826] Require Java 7#7rfscholte merged 4 commits intoapache:masterfrom mthmulders:MSHARED-826
Conversation
|
The OS related tests need to be changed, see https://builds.apache.org/job/maven-box/job/maven-shared-utils/job/MSHARED-826/test_results_analyzer/ |
|
@rfscholte |
|
@Tibor17 No, there's just a set of properties that are marked as static properties: they can only be set at startup and cannot be changed afterwards. https://github.com/AdoptOpenJDK/openjdk-jdk11/blob/master/src/java.base/share/classes/jdk/internal/util/StaticProperty.java shows a list. |
This way, we can first change the system props and then interact with the SUT. This way, changing the static class fields is no longer needed.
|
@mthmulders |
|
Thanks for the suggestion. I deliberately didn't want to mix the two (requiring Java 1.7 and using its new features). Java 7 is required for MSHARED-822 / #6, so we'll soon make use of at least some of the new features. |
|
Resolve #281 |
As suggested by @rfscholte in MSHARED-826: updated to maven-parent:33 and set property
javaVersionto 7.