RFR: 8314491: Linux: jexec launched via PATH fails to find java [v3]
Vladimir Petko
vpetko at openjdk.org
Mon Aug 21 22:52:53 UTC 2023
On Mon, 21 Aug 2023 21:36:13 GMT, David Holmes <dholmes at openjdk.org> wrote:
>>> Nit: no need to pre-declare this, just use `int error = getJavaPath(...)`
>>
>> This one was intentional to keep the style consistent =)
>
> The others are more widely used and there is a desire to document their meaning. That isn't necessary for this case, but as I said, just a nit.
Updated and re-run Jexec test.
==============================
Test summary
==============================
TEST TOTAL PASS FAIL ERROR
jtreg:test/jdk/tools/launcher/Jexec.java 1 1 0 0
==============================
TEST SUCCESS
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/15343#discussion_r1300725435
More information about the core-libs-dev
mailing list