RFR: 8325567: jspawnhelper without args fails with segfault [v7]

[Magnus Ihse Bursie]

On Fri, 8 Mar 2024 09:19:43 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:

>> Yes indeed, it is used for all Unix OSes (that is, everything but Windows).
>
> I think what matters for this test test most is which platforms we build `jspawnhelper` for, and those platforms indeed are:
> 
> 
> ifeq ($(call isTargetOs, macosx aix linux), true)
>   $(eval $(call SetupJdkExecutable, BUILD_JSPAWNHELPER, \
> 
> 
> So I'd say we just add `(os.family == "mac")` here. I would find it a bit weird to ask for `os.family != "windows"`, which would implicitly rely on exhaustiveness of current os family types.

Hm, that is not ideal code in the makefile. `jspawnhelper` is called from `src/java.base/unix/classes/java/lang/ProcessImpl.java`, so it is relied upon for all Unix implementation. Granted, this is currently the same as the list "macosx aix linux", but it would be better to express the same logic in the makefile as in the code.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/18112#discussion_r1517516188