20:27 Greg Lewis (glewis) 

java/openjdk17: Fix the build on arm64

    4b11728

19:52 Michael Osipov (michaelo) 

java/openjdk*: Allow users to specify an alternative cacerts file

Similar to certificate management with certctl(8) users can use custom
enterprise CAs. These need to be assembled into a custom cacerts file.
Therefore, an admin can install a custom truststore to an arbitrary
location and have the OpenJDK port symlink to it by adding
OPENJDK_SYMLINK_CACERTS to make.conf.
E.g., OPENJDK_SYMLINK_CACERTS=/usr/local/etc/ssl/cacerts.

PR:		229329
Approved by:	jrm (mentor), otis (mentor), glewis
Differential Revision:	https://reviews.freebsd.org/D42720

    d901bf5

22:51 Greg Lewis (glewis) 

java/openjdk17: Update to 17.0.10

    eeca11d

21:00 Piotr Kubaj (pkubaj) 

java/openjdk17: drop ELFv1 support

    e2a8a2ac

00:37 Muhammad Moinur Rahman (bofh) 

*/*: Sunset 12.4-RELEASE/12-STABLE from ports tree

- Remove all references to defunct ARCH arm
- Remove all references to defunct ARCH sparc64
- Remove x11-drivers/xf86-video-sunffb which requires defunct sparc64
  ARCH
- Remove sysutils/afbinit requires defunct sparc64 ARCH
- Remove all references to bktr driver
- Remove all references to defunct FreeBSD_12
- Remove all references to OSVERSION/OSREL corresponding to 12
- Remove conditionals in Mk/Uses/cabal.mk
- Remove sparc reference from Mk/Uses/qt-dist.mk
- Remove BROKEN_sparc64/NOT_FOR_ARCH=sparc64
- Remove BROKEN_FreeBSD_12* from:
- Remove OpenSSL patches from:
- Remove conditional flags for OSVERSION >= 1300000 to fixed flags.
  Also move conditional flags for non sparc64/arm ARCH to fixed flags.

Reviewed by:	brooks, jbeich, rene, salvadore
Differential Revision: https://reviews.freebsd.org/D42068

    bbab7f5

00:15 Greg Lewis (glewis) 

java/openjdk17: Update to 17.0.9

    bc0e38d

17:13 Daniel Engberg (diizzy) 

*/*: Bump jpeg-turbo users treewide

New major version 3.0.0

    664a2fa

06:58 Greg Lewis (glewis) 

java/openjdk17: Update to 17.0.8

    d504e5d

17:28 Greg Lewis (glewis) 

java/openjdk17: Update to 17.0.7

    261cada

05:19 Greg Lewis (glewis) 

java/openjdk17: Fix the build on some aarch64 machines

* Disable CompressedClassPointers on aarch64.  This fixes the build
  on some aarch64 machines where it previously failed, e.g. RPI4,
  and doesn't seem to affect the build on others (AWS aarch64 hardware).

This is based on and uses a patch initially created by ronald@

PR:		260187
Tested by:	ronald@

    b39d592

06:06 Greg Lewis (glewis) 

java/openjdk17: Update to 17.0.6

    15284be

17:36 Dimitry Andric (dim) 

java/openjdk17: fix build with clang 15

During an exp-run for llvm 15 (see bug 265425), it turned out that
java/openjdk17 failed to build with clang 15:

 
/wrkdirs/usr/ports/java/openjdk17/work/jdk17u-jdk-17.0.5-8-1/src/java.base/unix/native/libnet/DefaultProxySelector.c:374:41:
error: passing arguments to a function without a prototype is deprecated in all
versions of C and is not supported in C2x [-Werror,-Wdeprecated-non-prototype]
      proxies = (*g_proxy_resolver_lookup)(resolver, uri, NULL, &error);
                                          ^
 
/wrkdirs/usr/ports/java/openjdk17/work/jdk17u-jdk-17.0.5-8-1/src/java.base/unix/native/libnet/DefaultProxySelector.c:393:63:
error: passing arguments to a function without a prototype is deprecated in all
versions of C and is not supported in C2x [-Werror,-Wdeprecated-non-prototype]
                                  (*g_network_address_parse_uri)(proxies[i], 0,
                                                                ^
 
/wrkdirs/usr/ports/java/openjdk17/work/jdk17u-jdk-17.0.5-8-1/src/java.base/unix/native/libnet/DefaultProxySelector.c:398:70:
error: passing arguments to a function without a prototype is deprecated in all
versions of C and is not supported in C2x [-Werror,-Wdeprecated-non-prototype]
                              phost = (*g_network_address_get_hostname)(conn);
                                                                       ^
 
/wrkdirs/usr/ports/java/openjdk17/work/jdk17u-jdk-17.0.5-8-1/src/java.base/unix/native/libnet/DefaultProxySelector.c:399:66:
error: passing arguments to a function without a prototype is deprecated in all
versions of C and is not supported in C2x [-Werror,-Wdeprecated-non-prototype]
                              pport = (*g_network_address_get_port)(conn);
                                                                   ^
 
/wrkdirs/usr/ports/java/openjdk17/work/jdk17u-jdk-17.0.5-8-1/src/java.base/unix/native/libnet/DefaultProxySelector.c:435:22:
error: passing arguments to a function without a prototype is deprecated in all
versions of C and is not supported in C2x [-Werror,-Wdeprecated-non-prototype]
          (*g_strfreev)(proxies);
                       ^

This is because clang 15 warns by default about functions without
arguments in their prototypes. Fix this by suppressing the warning.

PR:		268366
Approved by:	portmgr (tcberner)
MFH:		2022Q4

    4d093f4

06:30 Greg Lewis (glewis) 

java/openjdk17: Use the distributed cacerts

PR:		266723
Reported by:	Michael Osipov <michael.osipov@siemens.com>

    381f912

22:21 Greg Lewis (glewis) 

java/openjdk17: Update to 17.0.5

    0d0710f

23:06 Greg Lewis (glewis) 

java/openjdk17: Update to 17.0.4.1

    19899c4

21:10 Stefan Eßer (se) 

Add WWW entries to port Makefiles

It has been common practice to have one or more URLs at the end of the
ports' pkg-descr files, one per line and prefixed with "WWW:". These
URLs should point at a project website or other relevant resources.

Access to these URLs required processing of the pkg-descr files, and
they have often become stale over time. If more than one such URL was
present in a pkg-descr file, only the first one was tarnsfered into
the port INDEX, but for many ports only the last line did contain the
port specific URL to further information.

There have been several proposals to make a project URL available as
a macro in the ports' Makefiles, over time.

This commit implements such a proposal and moves one of the WWW: entries
of each pkg-descr file into the respective port's Makefile. A heuristic
attempts to identify the most relevant URL in case there is more than
one WWW: entry in some pkg-descr file. URLs that are not moved into the
Makefile are prefixed with "See also:" instead of "WWW:" in the pkg-descr
files in order to preserve them.

There are 1256 ports that had no WWW: entries in pkg-descr files. These
ports will not be touched in this commit.

The portlint port has been adjusted to expect a WWW entry in each port
Makefile, and to flag any remaining "WWW:" lines in pkg-descr files as
deprecated.

Approved by:		portmgr (tcberner)

    b7f0544

13:14 Greg Lewis (glewis) 

java/openjdk17: Update to 17.0.4

    f20eed2

20:09 Rene Ladan (rene) 

cleanup: remove expired versions of OpenJDK (12 through 16)

Adjust ports depending on expired versions of OpenJDK:
- biology/snpeff: 12+ -> 17+
- devel/RStudio: 12 -> 11
- www/closure-compiler: 13 -> 11+

Allow java/openjdk17 to use java/openjdk17-bootstrap on i386
too, this builds just fine on 13.1-i386

Remove jdk12-doc as it is no longer relevant.

Clean up Java version calculations in bsd.java.mk

Reviewed by: jwb, java (glewis)

Differential Revision: https://reviews.freebsd.org/D35280

    c7d1c24

02:06 Greg Lewis (glewis) 

java/openjdk17: Update to 17.0.3 GA

    49c485c

07:56 Dimitry Andric (dim) 

java/openjdk13 java/openjdk14 java/openjdk15 java/openjdk16 java/openjdk17
java/openjdk17-jre: fix build with clang 14

During an exp-run for llvm 14 (see bug 261742), it turned out that
java/openjdk13 and java/openjdk17 fail to build with clang 14 (but this
also affects openjdk14 through 16):

=== Output from failing command(s) repeated here ===
* For target hotspot_variant-server_libjvm_objs_serviceThread.o:
/wrkdirs/usr/ports/java/openjdk13/work/jdk13u-jdk-13.0.10-5-1/src/hotspot/share/runtime/serviceThread.cpp:133:15:
error: use of bitwise '|' with boolean operands
[-Werror,-Wbitwise-instead-of-logical]
      while (((sensors_changed = LowMemoryDetector::has_pending_requests()) |
             ~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/wrkdirs/usr/ports/java/openjdk13/work/jdk13u-jdk-13.0.10-5-1/src/hotspot/share/runtime/serviceThread.cpp:133:15:
note: cast one or both operands to int to silence this warning
/wrkdirs/usr/ports/java/openjdk13/work/jdk13u-jdk-13.0.10-5-1/src/hotspot/share/runtime/serviceThread.cpp:133:15:
error: use of bitwise '|' with boolean operands
[-Werror,-Wbitwise-instead-of-logical]
      while (((sensors_changed = LowMemoryDetector::has_pending_requests()) |
              ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/wrkdirs/usr/ports/java/openjdk13/work/jdk13u-jdk-13.0.10-5-1/src/hotspot/share/runtime/serviceThread.cpp:133:15:
note: cast one or both operands to int to silence this warning
/wrkdirs/usr/ports/java/openjdk13/work/jdk13u-jdk-13.0.10-5-1/src/hotspot/share/runtime/serviceThread.cpp:133:15:
error: use of bitwise '|' with boolean operands
[-Werror,-Wbitwise-instead-of-logical]
      while (((sensors_changed = LowMemoryDetector::has_pending_requests()) |
              ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/wrkdirs/usr/ports/java/openjdk13/work/jdk13u-jdk-13.0.10-5-1/src/hotspot/share/runtime/serviceThread.cpp:133:15:
note: cast one or both operands to int to silence this warning
/wrkdirs/usr/ports/java/openjdk13/work/jdk13u-jdk-13.0.10-5-1/src/hotspot/share/runtime/serviceThread.cpp:133:15:
error: use of bitwise '|' with boolean operands
[-Werror,-Wbitwise-instead-of-logical]
      while (((sensors_changed = LowMemoryDetector::has_pending_requests()) |
              ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   ... (rest of output omitted)

Although the warning is normally an indication of a potential problem,
in this case a comment just before the affected code explictly mentions
the reason for using bitwise '|' instead of logical '||':

      // Process all available work on each (outer) iteration, rather than
      // only the first recognized bit of work, to avoid frequently true early
      // tests from potentially starving later work.  Hence the use of
      // arithmetic-or to combine results; we don't want short-circuiting.

(See
<https://github.com/openjdk/jdk/blob/master/src/hotspot/share/runtime/serviceThread.cpp#L140>)

Therefore, we should suppress -Wbitwise-instead-of-logical for clang 14
and higher.

PR:		262845
Approved by:	portmgr (build fix blanket)
MFH:		2022Q2

    e406118

04:41 Greg Lewis (glewis) 

java/openjdk17: Update to 17.0.2 GA

    57a7e70

02:43 Jung-uk Kim (jkim) 

java/openjdk17-jre: Add JRE-only package for java/openjdk17

    701f037

09:09 Greg Lewis (glewis) 

java/openjdk17: Add i386 support

    4807e7f

12:45 Piotr Kubaj (pkubaj) 

java/openjdk17: allow bootstrapping on powerpc64le with bootstrap-openjdk17

    69cdd4b

01:56 Piotr Kubaj (pkubaj) 

java/openjdk17: use java/bootstrap-openjdk17 on powerpc64 elfv2

    9089fb0

16:50 Greg Lewis (glewis) 

java/openjdk17: Use the bootstrap port on supported ARCHs

    11b6514

18:07 Greg Lewis (glewis) 

java/openjdk17: Update to 17.0.1

    a9d52e3

12:22 Dimitry Andric (dim) 

java/openjdk*: work around UB in markOopDesc, fix builds with clang 13

During an exp-run for llvm 13 (see bug 258209), it turned out that
java/openjdk11 through openjdk13 fail to build with clang 13:

=== Output from failing command(s) repeated here ===
* For target jdk__packages_attribute.done:

These crashes are all caused by the markOop/markOopDesc classes, which
are used to keep track of objects, and which are 'marked' using the low
few bits. (See
https://github.com/openjdk/jdk13u/blob/master/src/hotspot/share/oops/markOop.hpp
).

After some laborious bisecting, I found out that these crashes start
occuring after the upstream commit https://github.com
/llvm/llvm-project/commit/16d03818412 (Return "[CGCall] Annotate this
argument with alignment").

What happens afterwards, is that clang considers the "this" pointer to
always be aligned to the alignment of the actual object, and then
masking or adding a few low bits is not working as expected.

The reason openjdk14 and higher work fine with clang 13, and don't crash
similarly, is that the OpenJDK people completely redid the
markOop/markOopDesc classes in
https://github.com/openjdk/jdk/commit/ae5615c6142a4dc0d9033462f4880d7b3c127e26
("8229258: Rework markOop and markOopDesc into a simpler mark word value
carrier"). E.g, the markOopDesc class was renamed to markWord, and
*stores* a pointer-like value instead of *being* a pointer-like value.
This is a much safer way of handling things.

However, this upstream commit is *very* large, as are a few of its
follow-ups, which is probably the reason why it has not been backported
to JDKs <= 13. I tried manually backporting it, but got lost in many
nasty patch conflicts and problems.

As a workaround, build openjdk8 through 13 with clang12 from the
devel/llvm12 port, for the time being.

In addition, allow openjdk14 through 17 to be built with clang 13, by
adding -Wno-unused-but-set-parameter to the compilation flags.

PR:		258954
Approved by:	maintainer timeout (2 weeks)
MFH:		2021Q4

    3822416

00:28 Greg Lewis (glewis) 

java/openjdk17: Fix libfontmanager linkage

* Backport the fix from jdk17u so that --with-harfbuzz=system adds
  -lharfbuzz when linking libfontmanager.so

Reported by:	barulheira@gmail.com

    c1b344b

12:00 Mikael Urankar (mikael) 

java/openjdk17: Remove enable-aot configure argument.

The Experimental AOT and JIT Compiler were remove in OpenJDK17:
https://openjdk.java.net/jeps/410

Remove this configure argument as it causes build failure on aarch64:
configure: error: unrecognized options: --enable-aot

Approved by:	portmgr (build fix blanket)

    9c7b32b

22:19 Piotr Kubaj (pkubaj) 

java/openjdk17: enable on powerpc64

While here, sort ONLY_FOR_ARCHS.

    225ed0a

21:55 Piotr Kubaj (pkubaj) 

java/openjdk17: port to powerpc64le

    670c3c8

18:28 Greg Lewis (glewis) 

java/openjdk17: Add a port of OpenJDK 17

* Add a port of OpenJDK 17
* Restrict to amd64 and aarch64 until more platforms are tested

    1b1bfba