On Thu, 31 Aug 2023 15:08:34 GMT, Roger Riggs <[email protected]> wrote:
>> The problem with the environment variables is, that jtreg only passes very
>> few of them down to the testee process - USERDOMAIN and USERNAME are not
>> part of these as far as I know.
>
> ok, more overhead than value in the suggestion.
> If its windows, strip off the domain (before "/") and compare.
> If that doesn't work then just drop the username compare on windows.
After verifying that System.getenv yields empty results for USERDOMAIN and
USERNAME, I updated the change to use System.getProperty("user.name") in the
Windows Administrators case. Let's see how testing goes.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/15222#discussion_r1312738642
