Re: RFR: 8246112: Remove build-time and run-time checks for clock_gettime and CLOCK_MONOTONIC [v3]

2021-01-19 Thread Erik Joelsson
On Tue, 19 Jan 2021 01:53:03 GMT, David Holmes wrote: >> We are now confident that we have build-time and runtime support for >> clock_gettime and CLOCK_MONOTONIC on all our OpenJDK supported POSIX >> platforms - see bug report for some more details on different OS. >> Consequently we can

Re: RFR: 8253866: Security Libs Terminology Refresh

2021-01-14 Thread Erik Joelsson
On Thu, 14 Jan 2021 06:32:37 GMT, Jamil Nimeh wrote: > This is the security libs portion of the effort to replace > archaic/non-inclusive words with more neutral terms (see JDK-8253315 for > details). > > Here are the changes covering core libraries code and tests. Terms were > changed as

Re: RFR: 8259485: Document need for short paths when building on Windows

2021-01-12 Thread Erik Joelsson
On Tue, 12 Jan 2021 06:46:33 GMT, liach wrote: > Though this content seems simple, it would be extremely frustrating to > newcomers, especially when the build stalls at "cannot find valid visual > studio installation" for no clear reason in logs at all. Marked as reviewed by erikj

Re: RFR: 8259343: [macOS] Update JNI error handling in Cocoa code. [v3]

2021-01-11 Thread Erik Joelsson
On Mon, 11 Jan 2021 19:27:12 GMT, Phil Race wrote: >> Proposed updates to JNI error handling. > > Phil Race has updated the pull request incrementally with one additional > commit since the last revision: > > 8259343: [macOS] Update JNI error handling in Cocoa code. Build changes look good.

Re: RFR: 8259343: [macOS] Update JNI error handling in Cocoa code. [v2]

2021-01-11 Thread Erik Joelsson
On Mon, 11 Jan 2021 17:52:19 GMT, Phil Race wrote: >> Proposed updates to JNI error handling. > > Phil Race has updated the pull request incrementally with one additional > commit since the last revision: > > 8259343: [macOS] Update JNI error handling in Cocoa code.

Re: RFR: 8259559: COMPARE_BUILD can't compare patch files

2021-01-11 Thread Erik Joelsson
On Mon, 11 Jan 2021 11:11:01 GMT, Magnus Ihse Bursie wrote: > COMPARE_BUILD will fail to run the compare stage, since re-applying an > applied patch does not work. Marked as reviewed by erikj (Reviewer). - PR: https://git.openjdk.java.net/jdk/pull/2025

Re: RFR: 8258925: configure script failed on WSL [v5]

2021-01-11 Thread Erik Joelsson
On Sat, 9 Jan 2021 14:05:10 GMT, Yasumasa Suenaga wrote: >> I ran configure script on WSL 1, but it failed as below: >> >> $ bash configure --enable-debug --with-boot-jdk=/mnt/d/Java/jdk-15.0.1 >> >> : >> >> configure: Found potential Boot JDK using configure arguments >> configure: The

[jdk16] Integrated: JDK-8259429: Update reference to README.md

2021-01-08 Thread Erik Joelsson
On Thu, 7 Jan 2021 22:01:51 GMT, Erik Joelsson wrote: > In JDK-8251551, the top level README file changed names to README.md. In > jib-profiles.js we have a reference to this file to identify if the current > source tree is likely to be complete. This reference needs to be updated.

Re: RFR: 8258925: configure script failed on WSL [v3]

2021-01-08 Thread Erik Joelsson
On Fri, 8 Jan 2021 14:49:12 GMT, Yasumasa Suenaga wrote: >> I initially wanted to leave this for Magnus to look at since he wrote all of >> this, and I know he put a lot of effort into fixpath.sh. It's not a simple >> script. Now I have stared at it for a while, I think I understand the >>

Re: RFR: 8258925: configure script failed on WSL [v3]

2021-01-08 Thread Erik Joelsson
On Fri, 8 Jan 2021 13:07:02 GMT, David Holmes wrote: >> Yasumasa Suenaga has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Refactoring > > Hi Yasumasa, > > Okay I see the problem case now, and the latest fix seems to fix things in a >

Re: Adding to building info on space in windows directories

2021-01-08 Thread Erik Joelsson
I agree this would be useful and filed https://bugs.openjdk.java.net/browse/JDK-8259485 /Erik On 2021-01-07 21:35, - wrote: Hello, This is a followup of https://mail.openjdk.java.net/pipermail/build-dev/2021-January/029905.html. In that thread, my build on Windows 10 (not WSL) failed because

[jdk16] RFR: JDK-8259429: Update reference to README.md

2021-01-07 Thread Erik Joelsson
In JDK-8251551, the top level README file changed names to README.md. In jib-profiles.js we have a reference to this file to identify if the current source tree is likely to be complete. This reference needs to be updated. - Commit messages: - Change reference in jib-profiles.js

[jdk16] Integrated: JDK-8258657: Doc build is broken by use of new language features

2021-01-07 Thread Erik Joelsson
On Tue, 5 Jan 2021 22:51:29 GMT, Erik Joelsson wrote: > This patch changes how the docs-reference make target behaves to better > support the requirements for it. This target is used to generate javadocs in > a more stable way between releases, so that they those docs are bett

Re: [jdk16] RFR: JDK-8258657: Doc build is broken by use of new language features [v2]

2021-01-07 Thread Erik Joelsson
ocs are built (but still with a separate set of > parameters). > > This fix needs to go into JDK 16 so that the docs-reference target there can > be used to generate diffs for JDK 17. Erik Joelsson has updated the pull request incrementally with one additional commit since the last

Re: RFR: 8258143: Update --release 16 symbol information for JDK 16 build 30 or later

2021-01-06 Thread Erik Joelsson
On Wed, 6 Jan 2021 02:19:51 GMT, Joe Darcy wrote: > Updating to the symbol files for JDK 16 b30; change generated with the script > > make/scripts/generate-symbol-data.sh > > The change to the java.desktop module looks to be for JDK-8258373. Marked as reviewed by erikj (Reviewer).

[jdk16] RFR: JDK-8258657: Doc build is broken by use of new language features

2021-01-05 Thread Erik Joelsson
This patch changes how the docs-reference make target behaves to better support the requirements for it. This target is used to generate javadocs in a more stable way between releases, so that they those docs are better suited for generating diffs between releases. Currently we use the boot JDK

Re: Build and test problems on Windows with gtest

2021-01-04 Thread Erik Joelsson
Hello Matthias, The path configure found to vcruntime140_1.dll certainly looks strange. This is likely a bug in the build system. Normally we would expect to find this file in the redist dir of the Visual Studio installation. Does your community edition not include that file? /Erik On

Re: RFR: 8259045: Exception message from saproc.dll is garbled on Windows with Japanese locale

2021-01-04 Thread Erik Joelsson
On Mon, 4 Jan 2021 09:25:55 GMT, Yasumasa Suenaga wrote: > I got garbled exception message as following when I run `livenmethods` CLHSDB > command: > > sun.jvm.hotspot.debugger.DebuggerException : ?w???W > > My Windows laptop is set Japanese Locale, garbled message was written in >

Re: [jdk16] RFR: 8259043: More Zero architectures need linkage with libatomic

2021-01-04 Thread Erik Joelsson
On Mon, 4 Jan 2021 08:34:02 GMT, Aleksey Shipilev wrote: > JDK-8256831 added the Zero linkage with libatomic for MIPS, but there are > other 32-bit Zero platforms that need the same kind of treatment. > > Additional testing: > - [x] linux-mipsel-zero-fastdebug build Marked as reviewed by

Re: Cross-compilation of OpenJDK Zero with clang

2021-01-04 Thread Erik Joelsson
Hello Andy, Sorry for the late reply, but a lot of us have been on holiday. I recommend starting with the main documentation in the source tree at doc/building.md. I have no personal experience using Clang for cross compilation, but I would imagine it would function similar enough to GCC. To

Re: RFR: 8257733: Move module-specific data from make to respective module [v4]

2021-01-04 Thread Erik Joelsson
On Tue, 15 Dec 2020 22:56:15 GMT, Magnus Ihse Bursie wrote: >> A lot (but not all) of the data in make/data is tied to a specific module. >> For instance, the publicsuffixlist is used by java.base, and fontconfig by >> java.desktop. (A few directories, like mainmanifest, is *actually* used by

Re: RFR: 8258445: Move make/templates to make/data

2021-01-04 Thread Erik Joelsson
On Tue, 15 Dec 2020 23:15:23 GMT, Magnus Ihse Bursie wrote: > The `templates` directory in `make` is an odd bird. It actually contains data > files that the license checker needs, so it should move to `make/data`. Marked as reviewed by erikj (Reviewer). - PR:

Re: RFR: 8258449: Move make/hotspot/symbols to make/data

2021-01-04 Thread Erik Joelsson
On Tue, 15 Dec 2020 23:38:02 GMT, Magnus Ihse Bursie wrote: > The "symbol" files in make/hotspot/symbols are a list of exported symbols > from libjvm. As such, they are an essential data source for the build system, > and they should reside in make/data, not mixed with the hotspot make source

Re: RFR: 8258426: Split up autoconf/version-numbers and move it to conf dir [v2]

2021-01-04 Thread Erik Joelsson
On Tue, 15 Dec 2020 21:17:14 GMT, Magnus Ihse Bursie wrote: >> The file `make/autoconf/version-numbers` is plagued by a two-fold problem: >> first of all, it's a configuration file, not a part of autoconf, so it >> should reside in `make/conf` instead of `make/autoconf`. >> >> Secondly,

Re: RFR: 8258407: Split up CompileJavaModules.gmk into make/modules/$M/Java.gmk

2021-01-04 Thread Erik Joelsson
On Tue, 15 Dec 2020 13:11:30 GMT, Magnus Ihse Bursie wrote: > Right now `CompileJavaModules.gmk` contains two different part: one part with > the functionality needed to compile a java module, and one part were all > special requirements for all modules are listed. > > The second part should

Re: RFR: 8258407: Split up CompileJavaModules.gmk into make/modules/$M/Java.gmk [v3]

2021-01-04 Thread Erik Joelsson
On Tue, 15 Dec 2020 14:40:34 GMT, Magnus Ihse Bursie wrote: >> Right now `CompileJavaModules.gmk` contains two different part: one part >> with the functionality needed to compile a java module, and one part were >> all special requirements for all modules are listed. >> >> The second part

Re: RFR: 8258005: JDK build fails with incorrect fixpath script [v2]

2020-12-10 Thread Erik Joelsson
On Thu, 10 Dec 2020 13:11:47 GMT, Magnus Ihse Bursie wrote: >> The Leaning Toothpick Syndrome[1] strikes back! >> >> In the fixpath shell script wrapper, we transform single `` to `\`, to make >> them correctly escaped. Unfortunately, the bas variable substitution pattern >> I used were

Re: RFR: 8257973: UTIL_LOOKUP_PROGS should only find executable files

2020-12-09 Thread Erik Joelsson
On Wed, 9 Dec 2020 14:08:32 GMT, Magnus Ihse Bursie wrote: > It turns out that UTIL_LOOKUP_PROGS does not verify that when it find a file > in the PATH that matches the given tool name, it accept it straight away. > Even if the file is a directory, or is not executable. Marked as reviewed by

Re: RFR: 8257905: Make fixpath.sh more liberal in accepting paths embedded in arguments [v2]

2020-12-08 Thread Erik Joelsson
On Tue, 8 Dec 2020 16:43:14 GMT, Magnus Ihse Bursie wrote: >> When calling a Windows tool that needs an argument on the form >> `-option/a:/home/ihse/myjdk/...`, fixpath gets confused and thinks >> `/a:/home/ihse/myjdk` is a unix path, notices that it does not exists, and >> stops trying to

Re: RFR: 8257905: Make fixpath.sh more liberal in accepting paths embedded in arguments [v2]

2020-12-08 Thread Erik Joelsson
On Tue, 8 Dec 2020 16:40:26 GMT, Magnus Ihse Bursie wrote: >> When calling a Windows tool that needs an argument on the form >> `-option/a:/home/ihse/myjdk/...`, fixpath gets confused and thinks >> `/a:/home/ihse/myjdk` is a unix path, notices that it does not exists, and >> stops trying to

Re: RFR: 8257733: Move module-specific data from make to respective module [v2]

2020-12-08 Thread Erik Joelsson
On 2020-12-08 00:30, Magnus Ihse Bursie wrote: On Tue, 8 Dec 2020 02:40:43 GMT, Mandy Chung wrote: I have reviewed all lines in the patch file with or near instances of `jdk.compiler` Hi Magnus, I see the motivation of moving these build files for better identification of ownership.

Re: RFR: 8257679: Improved unix compatibility layer in Windows build (winenv) [v7]

2020-12-07 Thread Erik Joelsson
On Sat, 5 Dec 2020 01:13:34 GMT, Magnus Ihse Bursie wrote: >> For the build to work on Windows, we need a unix compatibility layer (known >> as the "winenv" in the build system). This can be e.g. Cygwin or Msys. The >> build system then needs to adapt various aspect to get the build to work in

Re: RFR: 8257572: Deprecate the archaic signal-chaining interfaces: sigset and signal

2020-12-07 Thread Erik Joelsson
On Thu, 3 Dec 2020 04:34:52 GMT, David Holmes wrote: > The signal-chaining facility was introduced in JDK 1.4 nearly 20 years ago > and supported three different Linux signal API's: sigset, signal and > sigaction: > >

Re: RFR: 8257572: Deprecate the archaic signal-chaining interfaces: sigset and signal

2020-12-07 Thread Erik Joelsson
On 2020-12-04 21:22, David Holmes wrote: Hi Erik, On 4/12/2020 12:07 am, Erik Joelsson wrote: In general, I would prefer if a flag like this was just inlined in the SetupJdkLibrary call, as there are no complex conditionals for setting it. Looking a bit further, the variable

RFR: JDK-8257789: Fix incremental build of test-image and bundles

2020-12-04 Thread Erik Joelsson
When building the test-bundle incrementally, it always gets rebuilt. This is caused by the prepare-test-image target in TestImage.gmk, where we create a README file as part of a recipe for a PHONY target. This means that the README is always rebuilt, which in turn triggers downstream rebuilds.

Integrated: JDK-8257633: Missing -mmacosx-version-min=X flag when linking libjvm

2020-12-04 Thread Erik Joelsson
On Fri, 4 Dec 2020 14:44:49 GMT, Erik Joelsson wrote: > We have historically provided -mmacosx-version-min=10.9.0 flags when > compiling and linking all our native executables and libraries on Macosx. > That flag seems to have disappeared when linking libjvm. This is sometimes >

Re: RFR: 8257450: Start of release updates for JDK 17

2020-12-04 Thread Erik Joelsson
On Tue, 1 Dec 2020 06:22:51 GMT, Joe Darcy wrote: > Start of JDK 17 updates. Build changes look good and I'm happy there are so few of them! - Marked as reviewed by erikj (Reviewer). PR: https://git.openjdk.java.net/jdk/pull/1531

Re: RFR: 8257679: Improved unix compatibility layer in Windows build (winenv) [v5]

2020-12-04 Thread Erik Joelsson
On Fri, 4 Dec 2020 15:57:44 GMT, Jorn Vernee wrote: >> @magicus Thanks! But, now I get the same error as the GH action: >> https://github.com/magicus/openjdk-sandbox/runs/1498822974#step:11:80 >> >> Looks like `-a` is not supported in my shell. I can make the error go away >> locally with: >>

RFR: JDK-8257633: Missing -mmacosx-version-min=X flag when linking libjvm

2020-12-04 Thread Erik Joelsson
We have historically provided -mmacosx-version-min=10.9.0 flags when compiling and linking all our native executables and libraries on Macosx. That flag seems to have disappeared when linking libjvm. This is sometimes causing our macosx builds to not be able to run on older versions of macosx.

Re: RFR: 8257679: Improved unix compatibility layer in Windows build (winenv) [v4]

2020-12-04 Thread Erik Joelsson
On Fri, 4 Dec 2020 12:45:57 GMT, Magnus Ihse Bursie wrote: >> Ok, it seems to have been caused by `usr/local/bin` being added to my Cygwin >> PATH in .bash_profile. If I remove that I get a little further. >> >> But the same command fails later on a path with a space in it: >> >> fixpath:

Re: RFR: 8257679: Improved unix compatibility layer in Windows build (winenv) [v4]

2020-12-04 Thread Erik Joelsson
On Thu, 3 Dec 2020 22:39:00 GMT, Magnus Ihse Bursie wrote: >> make/common/JavaCompilation.gmk line 241: >> >>> 239: >>> 240: $$($1_JAVAC_SERVER_CONFIG): $$($1_CONFIG_VARDEPS_FILE) >>> 241:$(ECHO) portfile=$$($1_JAVAC_PORT_FILE) > $$@ >> >> Did you consider using WriteFile here?

Re: RFR: 8257733: Move module-specific data from make to respective module

2020-12-04 Thread Erik Joelsson
On Fri, 4 Dec 2020 14:03:08 GMT, Erik Joelsson wrote: >>> And I can certainly move jdwp.spec to java.base instead. >> >> If jdwp.spec has to move to the src tree then src/java.se is probably the >> most suitable home because Java SE specifies JDWP as an optiona

Re: RFR: 8257733: Move module-specific data from make to respective module

2020-12-04 Thread Erik Joelsson
On Fri, 4 Dec 2020 12:30:02 GMT, Alan Bateman wrote: >> And I can certainly move jdwp.spec to java.base instead. That's the reason I >> need input on this: All I know is that is definitely not the responsibility >> of the Build Group to maintain that document, and I made my best guess at >>

Integrated: JDK-8257547: Handle multiple prereqs on the same line in deps files

2020-12-03 Thread Erik Joelsson
On Tue, 1 Dec 2020 22:49:06 GMT, Erik Joelsson wrote: > After fixing JDK-8256810 and starting to look into backporting it, I > discovered more potential failing cases. Certain versions of GCC may > sometimes output multiple prerequisite files on the same line. I think the >

Re: RFR: 8257679: Improved unix compatibility layer in Windows build (winenv) [v2]

2020-12-03 Thread Erik Joelsson
On Thu, 3 Dec 2020 15:30:15 GMT, Magnus Ihse Bursie wrote: >> For the build to work on Windows, we need a unix compatibility layer (known >> as the "winenv" in the build system). This can be e.g. Cygwin or Msys. The >> build system then needs to adapt various aspect to get the build to work in

Re: RFR: 8257572: Deprecate the archaic signal-chaining interfaces: sigset and signal

2020-12-03 Thread Erik Joelsson
On Thu, 3 Dec 2020 04:34:52 GMT, David Holmes wrote: > The signal-chaining facility was introduced in JDK 1.4 nearly 20 years ago > and supported three different Linux signal API's: sigset, signal and > sigaction: > >

Re: RFR: JDK-8257547: Handle multiple prereqs on the same line in deps files [v4]

2020-12-02 Thread Erik Joelsson
s. > > When splitting lines, we need to make sure to not just split on any space. > Some compilers output the : of the rule with a leading space, and already > split lines will have a space before the backslash. Erik Joelsson has updated the pull request incrementally with one additiona

Re: RFR: 8257620: Do not use objc_msgSend_stret to get macOS version

2020-12-02 Thread Erik Joelsson
On Wed, 2 Dec 2020 22:00:55 GMT, Erik Joelsson wrote: >> The current intention is to be consistent with the min system version and >> it's currently set to 10.9. If libjvm.dylib gets a different value, then >> that would be a bug, but note that this could also var

Re: RFR: 8257620: Do not use objc_msgSend_stret to get macOS version

2020-12-02 Thread Erik Joelsson
On Wed, 2 Dec 2020 21:57:15 GMT, Erik Joelsson wrote: >> Interesting, I still able to run the build after this change on macOS >> 10.9.5. I use jdk image and there is no LC_VERSION_MIN_MACOSX in libjvm. >> libjli, libjava have one, and it's 10.9 > > The current inten

Re: RFR: 8257620: Do not use objc_msgSend_stret to get macOS version

2020-12-02 Thread Erik Joelsson
On Wed, 2 Dec 2020 21:32:46 GMT, Anton Kozlov wrote: >> I wonder if we should be "upping" that to something later. >> 10.9 is over 7 years old and has been out of support for what - 4 years ? > > Interesting, I still able to run the build after this change on macOS 10.9.5. > I use jdk image and

Re: RFR: JDK-8257547: Handle multiple prereqs on the same line in deps files [v2]

2020-12-02 Thread Erik Joelsson
On Wed, 2 Dec 2020 20:01:08 GMT, Tim Bell wrote: >> Erik Joelsson has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Added test > > test/make/TestFixDepsFile.gmk line 60: > >> 58: $(ECHO) "

Re: RFR: JDK-8257547: Handle multiple prereqs on the same line in deps files [v3]

2020-12-02 Thread Erik Joelsson
s. > > When splitting lines, we need to make sure to not just split on any space. > Some compilers output the : of the rule with a leading space, and already > split lines will have a space before the backslash. Erik Joelsson has updated the pull request incrementally with one additiona

Re: RFR: JDK-8257547: Handle multiple prereqs on the same line in deps files [v3]

2020-12-02 Thread Erik Joelsson
On Wed, 2 Dec 2020 17:41:54 GMT, Tim Bell wrote: >> Erik Joelsson has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Make newline splitting work with BSD tar to make tests pass on macosx > > As @magicus wr

Re: RFR: JDK-8257547: Handle multiple prereqs on the same line in deps files [v2]

2020-12-02 Thread Erik Joelsson
s. > > When splitting lines, we need to make sure to not just split on any space. > Some compilers output the : of the rule with a leading space, and already > split lines will have a space before the backslash. Erik Joelsson has updated the pull request incrementally with one additiona

RFR: JDK-8257547: Handle multiple prereqs on the same line in deps files

2020-12-01 Thread Erik Joelsson
After fixing JDK-8256810 and starting to look into backporting it, I discovered more potential failing cases. Certain versions of GCC may sometimes output multiple prerequisite files on the same line. I think the easiest way to handle this new issue is to split such lines. When splitting

Integrated: JDK-8256810: Incremental rebuild broken on Macosx

2020-11-30 Thread Erik Joelsson
On Sat, 21 Nov 2020 00:12:30 GMT, Erik Joelsson wrote: > After fixing JDK-8256751, I noticed that the fix-deps-file macro isn't > actually doing the right thing at all. It seems clang on Macosx is > inconsistent with outputting relative or absolute paths in the deps files, so > so

Re: RFR: 8256657: Add cross-compiled build for Windows+Arm64 to submit workflow [v2]

2020-11-30 Thread Erik Joelsson
On Mon, 23 Nov 2020 15:19:18 GMT, Bernhard Urban-Forster wrote: >> I think you need to wait for #1350, and then reconcile this patch with that >> refactoring. > > Thanks for all the comments. > >> - 40-50 min builds seem to be excessive for just the hotspot build, do >> youknow what exactly

Re: RFR: 8256541: Sort out what version of awk is used in the build system

2020-11-30 Thread Erik Joelsson
On Thu, 26 Nov 2020 20:58:38 GMT, Magnus Ihse Bursie wrote: > For historical reasons, there exists a variety of different implementations > of awk: awk (the original implementation), gawk (the GNU version), nawk (new > awk, iirc) and the lesser known mawk. > > Things are complicated by the

RFR: JDK-8256810: Incremental rebuild broken on Macosx

2020-11-20 Thread Erik Joelsson
After fixing JDK-8256751, I noticed that the fix-deps-file macro isn't actually doing the right thing at all. It seems clang on Macosx is inconsistent with outputting relative or absolute paths in the deps files, so some files end up with double WORKSPACE_ROOT in the paths. It's also not

Integrated: JDK-8256751: Incremental rebuild with precompiled header fails when touching a header file

2020-11-20 Thread Erik Joelsson
On Fri, 20 Nov 2020 14:35:27 GMT, Erik Joelsson wrote: > When building with --disable-absolute-paths-in-output and precompiled header > enabled (both of which are default in release builds), then incremental > builds of Hotspot fails if a header file is touched. > > This is cau

RFR: JDK-8256751: Incremental rebuild with precompiled header fails when touching a header file

2020-11-20 Thread Erik Joelsson
When building with --disable-absolute-paths-in-output and precompiled header enabled (both of which are default in release builds), then incremental builds of Hotspot fails if a header file is touched. This is caused by the deps file the compiler generates is now containing relative paths, but

Re: RFR: 8256240: Reproducible builds should turn on the "deterministic" flag for Visual Studio

2020-11-20 Thread Erik Joelsson
On Wed, 18 Nov 2020 10:35:58 GMT, Magnus Ihse Bursie wrote: > The microsoft toolchain now supports a new "deterministic" build mode. This > goes a long way towards building properly deterministic. Another important > feature is that with the deterministic build mode enabled, the -pathmap flag,

Re: RFR: 8256497: Zero: enable G1 and Shenandoah GCs

2020-11-18 Thread Erik Joelsson
On Tue, 17 Nov 2020 19:02:03 GMT, Aleksey Shipilev wrote: > Following the [JDK-8255796](https://bugs.openjdk.java.net/browse/JDK-8255796) > improvement that ditched the inline contiguous alloc use from Zero, we can > now rely on GC interface to hook the GCs properly. G1 and Shenandoah are a >

Integrated: JDK-8256501: libTestMainKeyWindow fails to build with Xcode 12.2

2020-11-18 Thread Erik Joelsson
On Tue, 17 Nov 2020 21:04:08 GMT, Erik Joelsson wrote: > In Xcode 12.2, the framework JavaVM was removed in the sysroot and all > (relevant) frameworks inside were just moved one step up so we still find > things like JavaNativeFoundation and JavaRuntimeSupport. There is however on

Re: RFR: 8256538: Fix annoying awk warning in configure for java versions

2020-11-18 Thread Erik Joelsson
On Wed, 18 Nov 2020 09:47:58 GMT, Magnus Ihse Bursie wrote: > The change from grep to awk in JDK-8244248 and further bug fixed in > JDK-8244756 still has invalid syntax. This causes some awk (most notably > gawk, the most commonly used) to complain: > > gawk: cmd. line:1: warning: regexp

RFR: JDK-8256501: libTestMainKeyWindow fails to build with Xcode 12.2

2020-11-17 Thread Erik Joelsson
In Xcode 12.2, the framework JavaVM was removed in the sysroot and all (relevant) frameworks inside were just moved one step up so we still find things like JavaNativeFoundation and JavaRuntimeSupport. There is however one test that currently links explicitly to JavaVM, which fails the build.

Re: RFR: 8248862: Implement Enhanced Pseudo-Random Number Generators

2020-11-17 Thread Erik Joelsson
On Tue, 17 Nov 2020 19:58:47 GMT, Jim Laskey wrote: > This PR is to introduce a new random number API for the JDK. The primary API > is found in RandomGenerator and RandomGeneratorFactory. Further description > can be found in the JEP https://openjdk.java.net/jeps/356 . It looks like you have

Re: RFR: 8256430: add linux-x64-optimized to regular testing [v2]

2020-11-17 Thread Erik Joelsson
On Tue, 17 Nov 2020 19:31:16 GMT, Igor Ignatyev wrote: >> Hi all, >> >> >> [8256414](https://bugs.openjdk.java.net/browse/JDK-8256414) / #1233 added >> similar profile to submit workflow, this patch defines `linux-x64-optimized` >> profile in jib-profile so it can be used by mach5 and added

Re: RFR: 8256216: Enable reproducible builds in jib-profiles [v3]

2020-11-17 Thread Erik Joelsson
On Tue, 17 Nov 2020 14:37:22 GMT, Magnus Ihse Bursie wrote: >> The time has come to actually turn on the support for reproducible builds by >> default in the jib profiles. > > Magnus Ihse Bursie has updated the pull request incrementally with one > additional commit since the last revision: >

Re: RFR: 8256393: Github Actions build on Linux should define OS and GCC versions

2020-11-16 Thread Erik Joelsson
On Mon, 16 Nov 2020 12:51:25 GMT, Robin Westberg wrote: > We should be more explicit about OS and compiler versions used in the GitHub > Actions builds, to avoid problems caused by unexpected changes to the > defaults. This patch changes the OS and GCC versions used from ubuntu-latest >

Re: RFR: 8256308: Send arguments to javac server in a config file [v6]

2020-11-16 Thread Erik Joelsson
On Mon, 16 Nov 2020 12:44:08 GMT, Magnus Ihse Bursie wrote: >> Currently, to use the javac server, a horrendously long command line option >> is created, looking like this: `--server:portfile=> portfile>:sjavac=`, where the sjavac command has >> had all spaces replaced by %20. Since Project

Re: RFR: 8256354: Github Action build on Windows should define OS and MSVC versions

2020-11-13 Thread Erik Joelsson
On Fri, 13 Nov 2020 17:26:28 GMT, Robin Westberg wrote: > We should be more explicit about OS and compiler versions used in the GitHub > Actions builds, to avoid problems caused by unexpected changes to the > defaults. This patch changes the OS and MSVC versions used from latest > (currently

Re: RFR: 8256308: Send arguments to javac server in a config file

2020-11-13 Thread Erik Joelsson
On Thu, 12 Nov 2020 22:13:37 GMT, Magnus Ihse Bursie wrote: > Currently, to use the javac server, a horrendously long command line option > is created, looking like this: `--server:portfile= portfile>:sjavac=`, where the sjavac command has > had all spaces replaced by %20. Since Project

Re: RFR: 8256308: Send arguments to javac server in a config file [v5]

2020-11-13 Thread Erik Joelsson
On Fri, 13 Nov 2020 01:08:09 GMT, Magnus Ihse Bursie wrote: >> Currently, to use the javac server, a horrendously long command line option >> is created, looking like this: `--server:portfile=> portfile>:sjavac=`, where the sjavac command has >> had all spaces replaced by %20. Since Project

Re: RFR: 8256277: Github Action build on macOS should define OS and Xcode versions

2020-11-13 Thread Erik Joelsson
On Fri, 13 Nov 2020 11:48:31 GMT, Robin Westberg wrote: > We should be more explicit about OS and compiler versions used in the GitHub > Actions builds, to avoid problems caused by unexpected changes to the > defaults. This patch changes the OS and Xcode versions used from latest > (currently

Re: RFR: 8251854: [macosx] Java forces the use of discrete GPU

2020-11-10 Thread Erik Joelsson
On Tue, 10 Nov 2020 08:19:13 GMT, Sergey Bylokhov wrote: > This is a review request for the bug particularly fixed some time ago: > https://mail.openjdk.java.net/pipermail/2d-dev/2015-May/005425.html > > In that review request it was found that the old fix does not work well in > all cases,

Re: RFR: 8256048: Incomplete gitignore setting for netbeans project [v3]

2020-11-09 Thread Erik Joelsson
On Mon, 9 Nov 2020 23:31:09 GMT, Jie Fu wrote: >> Hi all, >> >> The gitignore setting for netbeans project seems to be incomplete. >> Only nbproject/private/ [1] is ignored. >> >> But there are also other files under nbproject/, which should be ignored too. >> >>

Re: RFR: 8256048: Incomplete gitignore setting for netbeans project [v2]

2020-11-09 Thread Erik Joelsson
On Mon, 9 Nov 2020 15:25:28 GMT, Jie Fu wrote: >> .gitignore line 5: >> >>> 3: /.idea/ >>> 4: /.vscode/ >>> 5: /nbproject/ >> >> I believe the existing nbproject/private/ is meant to catch any private >> files created by someone opening any of the various nbproject directories >> that exist

Re: RFR: 8256048: Incomplete gitignore setting for netbeans project [v2]

2020-11-09 Thread Erik Joelsson
On Mon, 9 Nov 2020 18:02:32 GMT, Erik Joelsson wrote: >> Thanks @erikj79 for your review. >> Updated. >> >> Any comments? >> Thanks. > > You misunderstand me. Adding the following line is fine and will make git > ignore any nbproject dir in the root of th

Re: RFR: 8256048: Incomplete gitignore setting for netbeans project

2020-11-09 Thread Erik Joelsson
On Mon, 9 Nov 2020 12:07:37 GMT, Jie Fu wrote: > Hi all, > > The gitignore setting for netbeans project seems to be incomplete. > Only nbproject/private/ [1] is ignored. > > But there are also other files under nbproject/, which should be ignored too. > > $ tree

Re: RFR: 8255895: Submit workflow artifacts miss hs_errs/replays due to ZIP include mismatch

2020-11-04 Thread Erik Joelsson
On Wed, 4 Nov 2020 13:43:58 GMT, Aleksey Shipilev wrote: >> Marked as reviewed by erikj (Reviewer). > > Thank you, @erikj79. Trivial? (I would like to get is soon, in order to > capture more hs_errs for current failures) I'm fine with a quick integration of this. - PR:

Re: RFR: 8255895: Submit workflow artifacts miss hs_errs/replays due to ZIP include mismatch

2020-11-04 Thread Erik Joelsson
On Wed, 4 Nov 2020 12:59:24 GMT, Aleksey Shipilev wrote: > Current submit workflow omits `hs_err*.log` and `replay*.log` files, because > zip matches the paths, and so only the hs_errs/replays in the current dir are > picked up (and there are none). It should be preceded with `*` to match

Re: RFR: 8253892: Disable misleading-indentation on clang as well as gcc

2020-11-04 Thread Erik Joelsson
On Tue, 3 Nov 2020 22:25:08 GMT, Magnus Ihse Bursie wrote: > With clang 10.0, the compiler now detects a new class of warnings. The > `misleading-indentation` warning has previously been disabled on gcc for > hotspot and libfdlibm. Now we need to disable it for clang as well. Marked as

Integrated: JDK-8255850: Hotspot recompiled on first incremental build

2020-11-03 Thread Erik Joelsson
On Tue, 3 Nov 2020 20:06:35 GMT, Erik Joelsson wrote: > After building the JDK from clean, the first incremental build of hotspot > will recompile all of it. This is caused by a difference in the CFLAGS > generated on the second go. The difference is generated in > JdkNativeCom

Re: RFR: 8255853: Update all nroff manpages for JDK 16 release

2020-11-03 Thread Erik Joelsson
On Tue, 3 Nov 2020 20:43:01 GMT, Magnus Ihse Bursie wrote: > The man pages in src/$module/man/*.1 is generated from the (unfortunately > still) closed markdown sources. > > These needs to be updated for JDK 16. During this updating, a typo in the GPL > header is also fixed (which caused

RFR: JDK-8255850: Hotspot recompiled on first incremental build

2020-11-03 Thread Erik Joelsson
After building the JDK from clean, the first incremental build of hotspot will recompile all of it. This is caused by a difference in the CFLAGS generated on the second go. The difference is generated in JdkNativeCompilation.gmk, where the module specific java header dir is always added to the

Re: RFR: 8255801: Race when building ct.sym build tools

2020-11-03 Thread Erik Joelsson
On Tue, 3 Nov 2020 13:39:19 GMT, Magnus Ihse Bursie wrote: > There is a race when compiling build.tools.symbolgenerator.CreateSymbols: > > [2020-11-03T07:21:12,118Z] Error: LinkageError occurred while loading main > class build.tools.symbolgenerator.CreateSymbols > [2020-11-03T07:21:12,119Z]

Re: RFR: 8251549: Update docs on building for Git

2020-11-03 Thread Erik Joelsson
On Tue, 3 Nov 2020 13:21:24 GMT, Michael Keck wrote: >> @erikj79 >> >> Ok, here is a completely rewritten portion, which highlights the differences >> between Cygwin git and Git for Windows. >> >> Suggestion: >> >> * You need to install a git client. You have two choices, Cygwin git

Re: RFR: 8255798: Remove dead headless code in CompileJavaModules.gmk

2020-11-03 Thread Erik Joelsson
On Tue, 3 Nov 2020 08:21:22 GMT, Magnus Ihse Bursie wrote: > The variable BUILD_HEADLESS_ONLY is no longer set. And sun/applet does not > exist anymore. Marked as reviewed by erikj (Reviewer). - PR: https://git.openjdk.java.net/jdk/pull/1031

Integrated: JDK-8255732: OpenJDK fails to build if $A is set to a value with spaces

2020-11-02 Thread Erik Joelsson
On Mon, 2 Nov 2020 14:40:35 GMT, Erik Joelsson wrote: > We have at least one java file with a '$' in the name. As we have learned > over the years, having $ in unexpected places quickly leads to unexpected > behavior in a shell/make based build. In this case it's our override &g

Integrated: JDK-8255673: Wrong version in docs bundles

2020-11-02 Thread Erik Joelsson
On Fri, 30 Oct 2020 17:19:56 GMT, Erik Joelsson wrote: > In JDK-8206311, when I introduced a new docs build profile, I forgot to add > the version configure arguments to it. This causes the docs bundles for > CI/promoted builds to be generated with the default adhoc version in

RFR: JDK-8255732: OpenJDK fails to build if $A is set to a value with spaces

2020-11-02 Thread Erik Joelsson
We have at least one java file with a '$' in the name. As we have learned over the years, having $ in unexpected places quickly leads to unexpected behavior in a shell/make based build. In this case it's our override mechanism of java files that needs protection against expanding such

Re: RFR: 8212879: Make JVMTI TagMap table not hash on oop address

2020-10-30 Thread Erik Joelsson
On Fri, 30 Oct 2020 20:23:04 GMT, Coleen Phillimore wrote: > This change turns the HashTable that JVMTI uses for object tagging into a > regular Hotspot hashtable - the one in hashtable.hpp with resizing and > rehashing. Instead of pointing directly to oops so that GC has to walk the >

RFR: JDK-8255673: Wrong version in docs bundles

2020-10-30 Thread Erik Joelsson
In JDK-8206311, when I introduced a new docs build profile, I forgot to add the version configure arguments to it. This causes the docs bundles for CI/promoted builds to be generated with the default adhoc version information instead of the specific version for that build. -

Integrated: JDK-8255620: Build race between modulegraphs and exploded-image-optimize targets

2020-10-30 Thread Erik Joelsson
On Thu, 29 Oct 2020 22:00:52 GMT, Erik Joelsson wrote: > When I reorganized top level docs build targets in JDK-8206311, a race was > introduced. The docs-*-modulegraph targets use the BUILD_JDK to run a build > tool. For this to work, the BUILD_JDK needs to be ready for usage. In a

Integrated: JDK-8255612: Explicitly disable dtrace for Oracle OpenJDK Linux builds

2020-10-30 Thread Erik Joelsson
On Thu, 29 Oct 2020 20:53:00 GMT, Erik Joelsson wrote: > The OpenJDK build on Linux will determine if the JVM feature "dtrace" should > be enabled based on the availability of dtrace on the build host. For Oracle > produced OpenJDK builds, we implicitly had this disable

RFR: JDK-8255620: Build race between modulegraphs and exploded-image-optimize targets

2020-10-29 Thread Erik Joelsson
When I reorganized top level docs build targets in JDK-8206311, a race was introduced. The docs-*-modulegraph targets use the BUILD_JDK to run a build tool. For this to work, the BUILD_JDK needs to be ready for usage. In a normal native build, this means that the exploded-image-optimize target

RFR: JDK-8255612: Explicitly disable dtrace for Oracle OpenJDK Linux builds

2020-10-29 Thread Erik Joelsson
The OpenJDK build on Linux will determine if the JVM feature "dtrace" should be enabled based on the availability of dtrace on the build host. For Oracle produced OpenJDK builds, we implicitly had this disabled because our build environment didn't have it installed. Rather than having to trust

Re: RFR: 8250669: Running JMH micros is broken after JDK-8248135

2020-10-28 Thread Erik Joelsson
On Wed, 28 Oct 2020 20:50:49 GMT, Claes Redestad wrote: > A microbenchmark with --enable-preview was added in JDK-8248135. This had the > unintended side effect that we now had to always run with --enable-preview. > > With records out of preview, I think the best course of action now is to

Re: RFR: 8255530: Additional cleanup after JDK-8235710 (elliptic curve removal)

2020-10-28 Thread Erik Joelsson
On Wed, 28 Oct 2020 18:42:01 GMT, Magnus Ihse Bursie wrote: > The fix for JDK-8235710 (Remove the legacy elliptic curves) unfortunately did > not remove all code related to elliptic curves in the build system. Marked as reviewed by erikj (Reviewer). - PR:

<    1   2   3   4   5   6   7   8   9   10   >