RFR: 8278422: Replace use of deprecated single string variant of Runtime.exec method

Nir Lisker nlisker at openjdk.org
Tue May 9 18:33:22 UTC 2023


On Tue, 9 May 2023 18:26:02 GMT, Andy Goryachev <angorya at openjdk.org> wrote:

>> Don't think we have an enum (maybe we should), but `PlatformUtil` already takes care of identifying the OS, also using `System.getProperty("os.name")`. You can use `PlatformUtil.isMac()`. If we make an enum, that's where is should be.
>
> There was a recent PR
> https://github.com/openjdk/jdk/pull/13357
> adding an enum deep in the jdk bowels
> 
> we probably can't use that

> There was a recent PR [openjdk/jdk#13357](https://github.com/openjdk/jdk/pull/13357) adding an enum deep in the jdk bowels
> 
> we probably can't use that

Isn't that for the architecture, not the OS?

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1130#discussion_r1188992367


More information about the openjfx-dev mailing list