Re: RFR: 8282493: Add --with-jcov-modules convenience option [v5]

2025-05-07 Thread Magnus Ihse Bursie
> Allow the user to set "--with-jcov-modules=java.desktop", instead of > "--with-jcov-filters=--include-modules java.desktop", since this is one of > the common use cases for the filtering. Magnus Ihse Bursie has updated the pull request incrementally with one additional commit since the last r

Re: RFR: 8282493: Add --with-jcov-modules convenience option [v5]

2025-05-07 Thread Magnus Ihse Bursie
On Wed, 7 May 2025 09:18:29 GMT, Magnus Ihse Bursie wrote: >> Allow the user to set "--with-jcov-modules=java.desktop", instead of >> "--with-jcov-filters=--include-modules java.desktop", since this is one of >> the common use cases for the filtering. > > Magnus Ihse Bursie has updated the pull

Re: RFR: 8282493: Add --with-jcov-modules convenience option [v5]

2025-05-07 Thread Magnus Ihse Bursie
On Wed, 7 May 2025 09:18:29 GMT, Magnus Ihse Bursie wrote: >> Allow the user to set "--with-jcov-modules=java.desktop", instead of >> "--with-jcov-filters=--include-modules java.desktop", since this is one of >> the common use cases for the filtering. > > Magnus Ihse Bursie has updated the pull

RFR: 8244533: Configure should abort on missing short names in Windows

2025-05-07 Thread Magnus Ihse Bursie
Windows can create short names for directories that contain spaces in the name. However, where short name creation was previously enabled by default, it no longer seems to be in recent version of Windows (the default setting is instead 'per volume', and the default for each volume is 'off'). We

Integrated: 8282493: Add --with-jcov-modules convenience option

2025-05-07 Thread Magnus Ihse Bursie
On Mon, 7 Apr 2025 08:45:13 GMT, Magnus Ihse Bursie wrote: > Allow the user to set "--with-jcov-modules=java.desktop", instead of > "--with-jcov-filters=--include-modules java.desktop", since this is one of > the common use cases for the filtering. This pull request has now been integrated. C

Integrated: 8356335: Remove linux-x86 from jib profiles

2025-05-07 Thread Magnus Ihse Bursie
On Wed, 7 May 2025 07:51:16 GMT, Magnus Ihse Bursie wrote: > After [JEP 503](https://openjdk.org/jeps/503), we cannot build linux-x86 so > it should not be included in the jib profiles. > > I also took the opportunity to remove the superfluous `--with-target-bits=64` > option, clean up some `

Re: RFR: 8356335: Remove linux-x86 from jib profiles [v2]

2025-05-07 Thread Erik Joelsson
On Wed, 7 May 2025 09:09:53 GMT, Magnus Ihse Bursie wrote: >> After [JEP 503](https://openjdk.org/jeps/503), we cannot build linux-x86 so >> it should not be included in the jib profiles. >> >> I also took the opportunity to remove the superfluous >> `--with-target-bits=64` option, clean up s

Re: RFR: 8282493: Add --with-jcov-modules convenience option [v5]

2025-05-07 Thread Erik Joelsson
On Wed, 7 May 2025 09:18:29 GMT, Magnus Ihse Bursie wrote: >> Allow the user to set "--with-jcov-modules=java.desktop", instead of >> "--with-jcov-filters=--include-modules java.desktop", since this is one of >> the common use cases for the filtering. > > Magnus Ihse Bursie has updated the pull

Integrated: 8356379: Need a proper way to test existence of binary from configure

2025-05-07 Thread Magnus Ihse Bursie
On Wed, 7 May 2025 10:57:27 GMT, Magnus Ihse Bursie wrote: > When we setup a command to run, e.g. $FOO we typically set this to just a > path to a binary, e.g. /usr/bin/foo. However, this is not necessarily true. > On Windows, this can be prefixed by the $FIXPATH prefix, and on all platforms >

Re: RFR: 8244533: Configure should abort on missing short names in Windows

2025-05-07 Thread Erik Joelsson
On Mon, 7 Apr 2025 09:18:08 GMT, Magnus Ihse Bursie wrote: > Windows can create short names for directories that contain spaces in the > name. > > However, where short name creation was previously enabled by default, it no > longer seems to be in recent version of Windows (the default setting

Re: RFR: 8356335: Remove linux-x86 from jib profiles [v2]

2025-05-07 Thread Magnus Ihse Bursie
> After [JEP 503](https://openjdk.org/jeps/503), we cannot build linux-x86 so > it should not be included in the jib profiles. > > I also took the opportunity to remove the superfluous `--with-target-bits=64` > option, clean up some `configure_args` constructs, and to reorder the > profiles to

Re: RFR: 8282493: Add --with-jcov-modules convenience option [v5]

2025-05-07 Thread Sergey Bylokhov
On Wed, 7 May 2025 11:22:40 GMT, Magnus Ihse Bursie wrote: > JAVA_TOOL_OPTIONS and _JAVA_OPTIONS are setup in lines 127...130 in > RunTests.gmk. > > @mrserb Is this expected behavior, or am I doing something wrong? That is a known issue: https://bugs.openjdk.org/browse/JDK-8219323 ---

RFR: 8356379: Need a proper way to test existence of binary from configure

2025-05-07 Thread Magnus Ihse Bursie
When we setup a command to run, e.g. $FOO we typically set this to just a path to a binary, e.g. /usr/bin/foo. However, this is not necessarily true. On Windows, this can be prefixed by the $FIXPATH prefix, and on all platforms we are allowed to pass arguments to the executable. If we want to t

Re: RFR: 8355003: Implement Ahead-of-Time Method Profiling [v14]

2025-05-07 Thread Chris Plummer
On Tue, 6 May 2025 21:50:34 GMT, Igor Veresov wrote: >> Improve warm-up time by making profile data from a previous run of an >> application instantly available, when the HotSpot Java Virtual Machine >> starts. Specifically, enhance the [AOT cache](https://openjdk.org/jeps/483) >> to store met

Re: RFR: 8356379: Need a proper way to test existence of binary from configure

2025-05-07 Thread Erik Joelsson
On Wed, 7 May 2025 10:57:27 GMT, Magnus Ihse Bursie wrote: > When we setup a command to run, e.g. $FOO we typically set this to just a > path to a binary, e.g. /usr/bin/foo. However, this is not necessarily true. > On Windows, this can be prefixed by the $FIXPATH prefix, and on all platforms >

Re: RFR: 8351029: IncludeCustomExtension does not work on cygwin with source code below /home

2025-05-07 Thread Erik Joelsson
On Wed, 7 May 2025 15:08:53 GMT, Magnus Ihse Bursie wrote: > If you check out your source code in Cygwin, somewhere under /home/... (as > opposed to /cygpath/...), IncludeCustomExtension does not work. > > The problem is that TOPDIR gets a different lexical value from the spec.gmk > file when

RFR: 8354278: Revert use of non-POSIX echo -n introduced in JDK-8301197

2025-05-07 Thread Magnus Ihse Bursie
`echo -n` is not part of the required POSIX standard, and built in shell versions of echo might not support it. Restore the previous use of `printf` instead. - Commit messages: - 8354278: Revert use of non-POSIX echo -n introduced in JDK-8301197 Changes: https://git.openjdk.org/jd

Re: RFR: 8354278: Revert use of non-POSIX echo -n introduced in JDK-8301197

2025-05-07 Thread Iris Clark
On Wed, 7 May 2025 21:25:39 GMT, Magnus Ihse Bursie wrote: > `echo -n` is not part of the required POSIX standard, and built in shell > versions of echo might not support it. Restore the previous use of `printf` > instead. Verified 15 uses of `$(ECHO) -n` introduced by [8301197](https://bugs.

Integrated: 8244533: Configure should abort on missing short names in Windows

2025-05-07 Thread Magnus Ihse Bursie
On Mon, 7 Apr 2025 09:18:08 GMT, Magnus Ihse Bursie wrote: > Windows can create short names for directories that contain spaces in the > name. > > However, where short name creation was previously enabled by default, it no > longer seems to be in recent version of Windows (the default setting

Re: RFR: 8334391: JDK build should exclude *-files directories for Java source

2025-05-07 Thread Erik Joelsson
On Fri, 4 Apr 2025 14:44:07 GMT, Magnus Ihse Bursie wrote: > We should always exclude `doc-files` and `snippet-files` when compiling java > code. To generalize this, exclude all `*-files` directories. > > (The usage of EXCLUDES and EXCLUDE_FILES is a bit weird. In essence, > EXCLUDE_FILES prep

Re: RFR: 8351029: IncludeCustomExtension does not work on cygwin with source code below /home

2025-05-07 Thread Magnus Ihse Bursie
On Wed, 7 May 2025 21:21:14 GMT, Erik Joelsson wrote: > Is it the `cd $$(TOPDIR)` when calling make from `DefineMakeTargets` that is > causing the mismatch that you are correcting by including SPEC here? No, it is the `-f $$(TOPDIR)/make/GenerateFindTests.gmk` argument to make. This is added v

Re: RFR: 8351029: IncludeCustomExtension does not work on cygwin with source code below /home

2025-05-07 Thread Magnus Ihse Bursie
On Wed, 7 May 2025 15:08:53 GMT, Magnus Ihse Bursie wrote: > If you check out your source code in Cygwin, somewhere under /home/... (as > opposed to /cygpath/...), IncludeCustomExtension does not work. > > The problem is that TOPDIR gets a different lexical value from the spec.gmk > file when

Re: RFR: 8351029: IncludeCustomExtension does not work on cygwin with source code below /home

2025-05-07 Thread Magnus Ihse Bursie
On Wed, 7 May 2025 21:17:47 GMT, Erik Joelsson wrote: >> If you check out your source code in Cygwin, somewhere under /home/... (as >> opposed to /cygpath/...), IncludeCustomExtension does not work. >> >> The problem is that TOPDIR gets a different lexical value from the spec.gmk >> file when

Re: RFR: 8351029: IncludeCustomExtension does not work on cygwin with source code below /home [v2]

2025-05-07 Thread Magnus Ihse Bursie
On Wed, 7 May 2025 22:50:52 GMT, Magnus Ihse Bursie wrote: >> If you check out your source code in Cygwin, somewhere under /home/... (as >> opposed to /cygpath/...), IncludeCustomExtension does not work. >> >> The problem is that TOPDIR gets a different lexical value from the spec.gmk >> file

Re: RFR: 8244533: Configure should abort on missing short names in Windows [v2]

2025-05-07 Thread Magnus Ihse Bursie
> Windows can create short names for directories that contain spaces in the > name. > > However, where short name creation was previously enabled by default, it no > longer seems to be in recent version of Windows (the default setting is > instead 'per volume', and the default for each volume i

Re: RFR: 8244533: Configure should abort on missing short names in Windows [v3]

2025-05-07 Thread Erik Joelsson
On Wed, 7 May 2025 20:26:13 GMT, Magnus Ihse Bursie wrote: >> Windows can create short names for directories that contain spaces in the >> name. >> >> However, where short name creation was previously enabled by default, it no >> longer seems to be in recent version of Windows (the default set

RFR: 8334391: JDK build should exclude *-files directories for Java source

2025-05-07 Thread Magnus Ihse Bursie
We should always exclude `doc-files` and `snippet-files` when compiling java code. To generalize this, exclude all `*-files` directories. (The usage of EXCLUDES and EXCLUDE_FILES is a bit weird. In essence, EXCLUDE_FILES prepends a `%` wildchar match to the filenames given, which we use here, w

Re: RFR: 8244533: Configure should abort on missing short names in Windows [v3]

2025-05-07 Thread Magnus Ihse Bursie
> Windows can create short names for directories that contain spaces in the > name. > > However, where short name creation was previously enabled by default, it no > longer seems to be in recent version of Windows (the default setting is > instead 'per volume', and the default for each volume i

RFR: 8351029: IncludeCustomExtension does not work on cygwin with source code below /home

2025-05-07 Thread Magnus Ihse Bursie
If you check out your source code in Cygwin, somewhere under /home/... (as opposed to /cygpath/...), IncludeCustomExtension does not work. The problem is that TOPDIR gets a different lexical value from the spec.gmk file when setup in Makefile, even if this was the same directory. Since THIS_INC

Re: RFR: 8351029: IncludeCustomExtension does not work on cygwin with source code below /home [v2]

2025-05-07 Thread Magnus Ihse Bursie
> If you check out your source code in Cygwin, somewhere under /home/... (as > opposed to /cygpath/...), IncludeCustomExtension does not work. > > The problem is that TOPDIR gets a different lexical value from the spec.gmk > file when setup in Makefile, even if this was the same directory. Since

Re: RFR: 8334391: JDK build should exclude *-files directories for Java source

2025-05-07 Thread Magnus Ihse Bursie
On Wed, 7 May 2025 21:12:54 GMT, Erik Joelsson wrote: >> We should always exclude `doc-files` and `snippet-files` when compiling java >> code. To generalize this, exclude all `*-files` directories. >> >> (The usage of EXCLUDES and EXCLUDE_FILES is a bit weird. In essence, >> EXCLUDE_FILES prep

Re: RFR: 8334391: JDK build should exclude *-files directories for Java source [v2]

2025-05-07 Thread Magnus Ihse Bursie
> We should always exclude `doc-files` and `snippet-files` when compiling java > code. To generalize this, exclude all `*-files` directories. > > (The usage of EXCLUDES and EXCLUDE_FILES is a bit weird. In essence, > EXCLUDE_FILES prepends a `%` wildchar match to the filenames given, which we >

Re: RFR: 8301971: Make JDK source code UTF-8 [v3]

2025-05-07 Thread Matthias Baesken
On Wed, 16 Apr 2025 10:35:02 GMT, Matthias Baesken wrote: >> Magnus Ihse Bursie has updated the pull request incrementally with three >> additional commits since the last revision: >> >> - Also document UTF-8 requirements (solves JDK-8338973) >> - Let configure only accept utf-8 locales >> -

RFR: 8356335: Remove linux-x86 from jib profiles

2025-05-07 Thread Magnus Ihse Bursie
After [JEP 503](https://openjdk.org/jeps/503), we cannot build linux-x86 so it should not be included in the jib profiles. I also took the opportunity to remove the superfluous `--with-target-bits=64` option, clean up some `configure_args` constructs, and to reorder the profiles to group all l

Re: RFR: 8301971: Make JDK source code UTF-8 [v5]

2025-05-07 Thread Matthias Baesken
On Tue, 6 May 2025 15:39:04 GMT, Magnus Ihse Bursie wrote: >> Most of the JDK code base has been transitioned to UTF-8, but not all. This >> has recently become an acute problem, since our mixing of iso-8859-1 and >> utf-8 in properties files confused the version of `sed` that is shipped with

Re: RFR: 8301971: Make JDK source code UTF-8 [v5]

2025-05-07 Thread Martin Doerr
On Tue, 6 May 2025 15:39:04 GMT, Magnus Ihse Bursie wrote: >> Most of the JDK code base has been transitioned to UTF-8, but not all. This >> has recently become an acute problem, since our mixing of iso-8859-1 and >> utf-8 in properties files confused the version of `sed` that is shipped with

Re: RFR: 8282493: Add --with-jcov-modules convenience option [v4]

2025-05-07 Thread Magnus Ihse Bursie
> Allow the user to set "--with-jcov-modules=java.desktop", instead of > "--with-jcov-filters=--include-modules java.desktop", since this is one of > the common use cases for the filtering. Magnus Ihse Bursie has updated the pull request with a new target base due to a merge or a rebase. The pu