[jdk17u-dev] Integrated: 8286781: Replace the deprecated/obsolete gethostbyname and inet_addr calls

[fitzsim]

On Tue, 21 May 2024 15:49:26 GMT, fitzsim <duke at openjdk.org> wrote:

> I propose backporting the fix for `8286781` to `jdk17u`.
> 
> I tested the changes on `jdk17u-dev` `master`, on `Fedora 38` `x86-64`, `slowdebug` and `fastdebug` configurations.  The test was to run `IdealGraphVisualizer`, per `jdk17u-dev/src/utils/IdealGraphVisualizer/README.md`, then run, e.g.:
> 
> 
> ./build/linux-x86_64-server-fastdebug/jdk/bin/java -XX:+PrintIdealGraph -XX:PrintIdealGraphLevel=1 -classpath ~/source/hello-java Hello
> 
> 
> I needed extra JVM arguments in `jdk17u-dev/src/utils/IdealGraphVisualizer/application/target/idealgraphvisualizer/bin/idealgraphvisualizer` to make the visualizer utility work on 17:
> 
> 
>             --jdkhome '"$jdkhome"' \
>             -J--add-opens=java.base/java.net=ALL-UNNAMED \
>             -J--add-opens=java.base/java.security=ALL-UNNAMED \
>             -J--add-opens=java.base/java.lang=ALL-UNNAMED \
>             -J--add-opens=java.desktop/sun.awt=ALL-UNNAMED \
>             -J--add-opens=java.desktop/javax.swing.plaf.synth=ALL-UNNAMED \
>             -J--add-opens=java.desktop/javax.swing=ALL-UNNAMED \
>             -J--add-exports=java.desktop/com.sun.java.swing.plaf.gtk=ALL-UNNAMED \
>             --clusters '"$clusters"' \
> 
> 
> Networking between the test and the utility worked fine with the backport applied.

This pull request has now been integrated.

Changeset: 4071b8c9
Author:    Thomas Fitzsimmons <fitzsim at redhat.com>
Committer: Andrew John Hughes <andrew at openjdk.org>
URL:       https://git.openjdk.org/jdk17u-dev/commit/4071b8c9be8e763d48bab1a5b8d57729a4f1307b
Stats:     39 lines in 5 files changed: 13 ins; 16 del; 10 mod

8286781: Replace the deprecated/obsolete gethostbyname and inet_addr calls

Reviewed-by: andrew
Backport-of: d7298245d6759f62e253b5cf0df975db17fdbf82

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

PR: https://git.openjdk.org/jdk17u-dev/pull/2487