Re: RFR: 8295343: sun/security/pkcs11 tests fail on Linux RHEL 8.6 and newer [v4]

2023-11-20 Thread Valerie Peng
On Tue, 24 Oct 2023 19:58:58 GMT, Matthew Donovan wrote: >> Hello, >> >> In this PR I removed NSS tests from the ProblemList and updated NssTest to >> use Sqlite databases. I also removed code in PKCS11Test.java that falls back >> to NSS libraries installed on the test system. The tests can st

Re: RFR: 8264425: Update building.md on non-English locales on Windows [v3]

2023-11-20 Thread Erik Joelsson
On Mon, 20 Nov 2023 15:08:22 GMT, Magnus Ihse Bursie wrote: >> Update the build documentation to reflect the fact that non-English locales >> are only supported on a "best effort" principle, and add instructions how to >> install the necessary language pack to Visual Studio. > > Magnus Ihse Bur

Re: RFR: 8308753: Class-File API transition to Preview [v29]

2023-11-20 Thread Adam Sotona
> Classfile API is an internal library under package `jdk.internal.classfile`  > in JDK 21. > This pull request turns the Classfile API into a preview feature and moves it > into `java.lang.classfile`. > It repackages all uses across JDK and tests and adds lots of missing Javadoc. > > This PR goe

Re: RFR: 8308753: Class-File API transition to Preview [v28]

2023-11-20 Thread Adam Sotona
> Classfile API is an internal library under package `jdk.internal.classfile`  > in JDK 21. > This pull request turns the Classfile API into a preview feature and moves it > into `java.lang.classfile`. > It repackages all uses across JDK and tests and adds lots of missing Javadoc. > > This PR goe

Re: RFR: 8318913: The module-infos for --release data do not contain pre-set versions [v2]

2023-11-20 Thread Jan Lahoda
On Fri, 10 Nov 2023 00:42:43 GMT, Magnus Ihse Bursie wrote: > The build changes almost look good. Just remove the extra added lines at the > end. > > I think this point too still remains: > > > 1. TransitiveDependencies seems to be unused now. I assume this is > > intended. But maybe the java

Re: RFR: 8264425: Update building.md on non-English locales on Windows [v3]

2023-11-20 Thread Naoto Sato
On Mon, 20 Nov 2023 15:08:22 GMT, Magnus Ihse Bursie wrote: >> Update the build documentation to reflect the fact that non-English locales >> are only supported on a "best effort" principle, and add instructions how to >> install the necessary language pack to Visual Studio. > > Magnus Ihse Bur

Re: RFR: 8306055: Add a built-in Catalog to JDK XML module [v3]

2023-11-20 Thread Lance Andersen
On Mon, 20 Nov 2023 17:46:53 GMT, Joe Wang wrote: >> Implement the built-in Catalog. > > Joe Wang has updated the pull request incrementally with one additional > commit since the last revision: > > add a note; fix alignment Thank you Joe. All good by me and Congrats on moving this across t

Re: RFR: 8306055: Add a built-in Catalog to JDK XML module [v2]

2023-11-20 Thread Joe Wang
On Mon, 20 Nov 2023 16:48:51 GMT, Lance Andersen wrote: > Hi Joe, > > Thank you for all of your hard work and perseverance to drive this project as > I realize it was a lot of work and iterations based on feedback from the team. > > Overall looks good. A couple of minor comments below Thanks

Re: RFR: 8306055: Add a built-in Catalog to JDK XML module [v2]

2023-11-20 Thread Joe Wang
On Mon, 20 Nov 2023 16:38:54 GMT, Magnus Ihse Bursie wrote: > Build change trivially okay Thanks Magnus! - PR Comment: https://git.openjdk.org/jdk/pull/16719#issuecomment-1819529895

Re: RFR: 8306055: Add a built-in Catalog to JDK XML module [v3]

2023-11-20 Thread Joe Wang
> Implement the built-in Catalog. Joe Wang has updated the pull request incrementally with one additional commit since the last revision: add a note; fix alignment - Changes: - all: https://git.openjdk.org/jdk/pull/16719/files - new: https://git.openjdk.org/jdk/pull/16719/fil

Re: RFR: 8318913: The module-infos for --release data do not contain pre-set versions [v4]

2023-11-20 Thread Jan Lahoda
> Consider a simple module, like: > > module test {} > > > And compile it with JDK 22 and JDK 21 using: > javac --release 21 > > The results of the compilations will differ: when compiling with JDK 21, the > mandated java.base dependency will get a version, possibly like > "21-internal". When

Re: RFR: 8306055: Add a built-in Catalog to JDK XML module [v2]

2023-11-20 Thread Lance Andersen
On Sun, 19 Nov 2023 23:36:16 GMT, Joe Wang wrote: >> Implement the built-in Catalog. > > Joe Wang has updated the pull request incrementally with one additional > commit since the last revision: > > remove J2SE directory; add note explaining how DTDs are resolved Hi Joe, Thank you for all o

Re: RFR: 8306055: Add a built-in Catalog to JDK XML module [v2]

2023-11-20 Thread Magnus Ihse Bursie
On Sun, 19 Nov 2023 23:36:16 GMT, Joe Wang wrote: >> Implement the built-in Catalog. > > Joe Wang has updated the pull request incrementally with one additional > commit since the last revision: > > remove J2SE directory; add note explaining how DTDs are resolved Build change trivially okay

Integrated: 8320410: Reflow markdown in building.md

2023-11-20 Thread Magnus Ihse Bursie
On Mon, 20 Nov 2023 14:34:51 GMT, Magnus Ihse Bursie wrote: > Reflow the markdown in building.md so line lengths of 80 is respected, > wherever possible. > > Note that is is correct that the building.html file is unchanged (and indeed > a sign that only markdown line breaks have been modified

Re: RFR: 8320410: Reflow markdown in building.md

2023-11-20 Thread Erik Joelsson
On Mon, 20 Nov 2023 14:34:51 GMT, Magnus Ihse Bursie wrote: > Reflow the markdown in building.md so line lengths of 80 is respected, > wherever possible. > > Note that is is correct that the building.html file is unchanged (and indeed > a sign that only markdown line breaks have been modified

RFR: 8320410: Reflow markdown in building.md

2023-11-20 Thread Magnus Ihse Bursie
Reflow the markdown in building.md so line lengths of 80 is respected, wherever possible. Note that is is correct that the building.html file is unchanged (and indeed a sign that only markdown line breaks have been modified). - Commit messages: - 8320410: Reflow markdown in build

Re: RFR: 8264425: Update building.md on non-English locales on Windows [v2]

2023-11-20 Thread Magnus Ihse Bursie
On Fri, 17 Nov 2023 13:06:53 GMT, Magnus Ihse Bursie wrote: >> Update the build documentation to reflect the fact that non-English locales >> are only supported on a "best effort" principle, and add instructions how to >> install the necessary language pack to Visual Studio. > > Magnus Ihse Bur

Re: RFR: 8264425: Update building.md on non-English locales on Windows [v3]

2023-11-20 Thread Magnus Ihse Bursie
> Update the build documentation to reflect the fact that non-English locales > are only supported on a "best effort" principle, and add instructions how to > install the necessary language pack to Visual Studio. Magnus Ihse Bursie has updated the pull request incrementally with one additional

Re: RFR: 8264425: Update building.md on non-English locales on Windows [v3]

2023-11-20 Thread Magnus Ihse Bursie
On Fri, 17 Nov 2023 18:27:44 GMT, Naoto Sato wrote: >> Oh, here are lots of news to me. >> >> 1) What is the system locale? How do you check which locale is the system >> locale (as opposed to the user locale), and how to you change it? >> >> 2) What is the UTF-8 beta option? > > IMHO, Windows

Re: RFR: 8295343: sun/security/pkcs11 tests fail on Linux RHEL 8.6 and newer [v2]

2023-11-20 Thread Magnus Ihse Bursie
On Mon, 23 Oct 2023 11:43:06 GMT, Matthew Donovan wrote: >> I am not sure, but shouldn`t the change also adopt the documentation in >> test/jdk/sun/security/pkcs11/README > >> I am not sure, but shouldn`t the change also adopt the documentation in >> test/jdk/sun/security/pkcs11/README > > Goo