On Tue, 11 Feb 2025 07:46:54 GMT, Daniel Jeliński <djelin...@openjdk.org> wrote:
> `make test TEST=gtest` fails to run gtest on WSL. This is because we're > looking for a nonexistent file `gtestLauncher`; on Windows the file is named > `gtestLauncher.exe`. > > This PR adds `$EXECUTABLE_SUFFIX` to the executable path, similar to what > [GtestImage.gmk already > does](https://github.com/openjdk/jdk/blob/64bd8d2592d26e02a7f2f96caa47cba5e158aaa2/make/hotspot/test/GtestImage.gmk#L37). > > I verified that with the proposed change, `make test TEST=gtest` works > correctly both on WSL and on Cygwin. Other platforms are unaffected, because > EXECUTABLE_SUFFIX is [empty on all non-Windows > platforms](https://github.com/openjdk/jdk/blob/64bd8d2592d26e02a7f2f96caa47cba5e158aaa2/make/autoconf/toolchain.m4#L188). Marked as reviewed by erikj (Reviewer). ------------- PR Review: https://git.openjdk.org/jdk/pull/23555#pullrequestreview-2608790106
