Build and run on Haiku

[duncanmcqueen]

Summary

This pull request lets GNU Classpath build and run on Haiku (x86_64). It has three small changes:

• configure.ac: Haiku has the socket functions and h_errno in libnetwork. Now AC_SEARCH_LIBS for inet_pton also looks in network.
• VMNetworkInterface: Haiku does not define IFF_RUNNING. The file now defines it if necessary.
• cpnet: On Haiku, gethostbyname_r() is in libnetwork, but no header declares it. Configure finds the function but not the number of arguments, and cpnet.c stops with #error. Now cpnet.c uses gethostbyname() in this case.

With the config.guess and config.sub from current automake, configure runs on Haiku as x86_64-unknown-haiku.

Tests

On Haiku R1/beta6 x86_64: autoreconf -fi, configure with --enable-jni --disable-gtk-peer --disable-gconf-peer --disable-plugin --disable-gjdoc --disable-examples --enable-default-preferences-peer=file --disable-tools, make and make install complete. I used the result with Kaffe 1.1.10. A command line test (including network and processes) passes.

AI assistance

An AI model (Claude Code, by Anthropic) wrote these changes and this text. The repository owner directed the work. Please review the changes as you review other contributions.

The commits do not have a Signed-off-by line. If you need one, the owner will add it.

🤖 Generated with Claude Code

[guillerodriguez]

Hi @duncanmcqueen, thanks for these too!

As with #59, could you split this into separate PRs, one per fix? Steps to reproduce or a small test case for each would be very helpful as well.

One general note: we’re a bit cautious with platform-specific changes, and prefer fixes that improve portability in general rather than ones that target a single OS. So when you open the new PRs, it would help if each one mentioned whether the issue is specific to Haiku or could possibly impact other systems.

Thanks!

[duncanmcqueen]

Thank you for the review. I split this pull request into one pull request for each fix:

• configure.ac: look for the socket functions in libnetwork too #73 configure.ac: look for the socket functions in libnetwork too
• cpnet: use gethostbyname_r() only when its declaration is known #74 cpnet: use gethostbyname_r() only when its declaration is known
• VMNetworkInterface: define IFF_RUNNING if the system does not #75 VMNetworkInterface: define IFF_RUNNING if the system does not

Each pull request tells if the problem is specific to Haiku:

• configure.ac: look for the socket functions in libnetwork too #73 is specific to Haiku (and BeOS). It has no effect on Linux, because AC_SEARCH_LIBS finds inet_pton in the C library there.
• cpnet: use gethostbyname_r() only when its declaration is known #74 is a general portability fix: it occurs on each system that has gethostbyname_r() in a library but does not declare it. The result does not change on Linux.
• VMNetworkInterface: define IFF_RUNNING if the system does not #75 is for systems that do not define IFF_RUNNING. It has no effect on systems that define it.

I reproduced each problem on Haiku with the master branch, and each pull request has the results before and after the change. The three changes depend on each other on Haiku: master does not build there without #75, and #73 shows the cpnet.c error that #74 fixes. Thus #74 must be merged before #73 or together with it.

While I wrote the test for #75, I found an error in this pull request: it defined IFF_RUNNING as 0x40, which is IFF_NOARP on Haiku. #75 uses Haiku's IFF_LINK instead.

I close this pull request, because the three new pull requests replace it.

🤖 Generated with Claude Code