[jdk21u-dev] RFR: 8361613: System.console() should only be available for interactive terminal

Goetz Lindenmaier goetz at openjdk.org
Wed Feb 4 11:45:22 UTC 2026


On Tue, 3 Feb 2026 13:20:51 GMT, Ralf Schmelter <rschmelter at openjdk.org> wrote:

>> Had to adapt change to JdkConsoleProviderImpl.java. Touched method looks quite different.
>> 
>> Omitted change to test/jdk/java/io/Console/DefaultCharsetTest.java
>> which was added by  "https://bugs.openjdk.org/browse/JDK-8341975: Unable to set encoding for IO.println, IO.print and IO.readln" in 24
>> Omitted change to test/jdk/java/io/Console/LocaleTest.java
>> which was added by "https://bugs.openjdk.org/browse/JDK-8330276: Console methods with explicit Locale" 23. Has CSR and Release Note.
>> Omitted change to test/jdk/java/lang/IO/IO.java
>> which was added by "https://bugs.openjdk.org/browse/JDK-8344706: Implement JEP 512: Compact Source Files and Instance Main Methods"
>> 
>> Trivially resolved ModuleSelectionTest.java.
>> 
>> Omitted change to test/jdk/jdk/internal/jline/JLineConsoleProviderTest.java
>> which was added by "https://bugs.openjdk.org/browse/JDK-8331535: Incorrect prompt for Console.readLine" in 23. Backport?
>> Omitted change to test/jdk/jdk/internal/jline/LazyJdkConsoleProvider.java
>> which was added by "https://bugs.openjdk.org/browse/JDK-8333086: Using Console.println is unnecessarily slow due to JLine initalization" in 23.
>> test/jdk/jdk/internal/jline/RedirectedStdOut.java cannot be deleted/moved
>> It was added by "https://bugs.openjdk.org/browse/JDK-8330998: System.console() writes to stderr when stdout is redirected" in 23.
>> 
>> I'm not sure what to do with files locale.exp and defaultCharset.exp.  They would be used by the omitted tests.
>
> test/jdk/java/io/Console/defaultCharset.exp line 1:
> 
>> 1: #
> 
> Does it make sense to downport this files, since it is only used by DefaultCharsetTest, which itself isn't downported?

Hi @schmelter-sap 
Yes, that's what I thought about.  I don't have a clear opinion.

-------------

PR Review Comment: https://git.openjdk.org/jdk21u-dev/pull/2592#discussion_r2763581421


More information about the jdk-updates-dev mailing list