RFR: 8353439: Shell grouping of -XX:OnError= commands is surprising

[David Holmes]

On Tue, 1 Apr 2025 10:59:16 GMT, Kevin Walls <kevinw at openjdk.org> wrote:

> We should be consistent, and run all OnError items in a new shell.  Currently the ; separator causes a new shell, but multiple -XX:OnError= options are grouped into the same shell.
> 
> next_OnError_command() decides on where a new command starts.  It should recognise newlines, and all commands will get their own shell.

I am having trouble understanding how the current behaviour can actually work. If I have

java -XX:OnError="gcore %p" ... -XX:OnError="ps -fe" ...

then we will get a combined `OnError` value of "gcore %p\nps -fe" - which as a single command is nonsense. ???

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

PR Comment: https://git.openjdk.org/jdk/pull/24354#issuecomment-2777325939