[jdk17u-dev] RFR: 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.

I have not tested this on Windows yet.  I did not realize testing is not configured on jdk17u-dev.  I will try to at least get the pre-submit tests run on my private repo (or some other way).

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

PR Comment: https://git.openjdk.org/jdk17u-dev/pull/2487#issuecomment-2122952070