RFR: 8319574: Exec/process tests should be marked as flagless
David Holmes
dholmes at openjdk.org
Thu Nov 9 07:42:57 UTC 2023
On Thu, 9 Nov 2023 06:43:12 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:
>> Tests that directly use ProcessBuilder to spawn processes and do not pass the standard test command line arguments should be marked as `vm.flagless` to indicate to the testing infrastructure that they do not accept them and should not be run with extra test command line options `test.vm.opts` and `test.java.opts`.
>>
>> Add @requires vm.flagless to tests
>
> The changes look OK to me.
>
> It wasn't clear to me what "flagless" meant. I checked the implementation of how this `@requires` is implemented https://github.com/openjdk/jdk/blob/master/test/jtreg-ext/requires/VMProps.java#L657. So essentially, the definition of "flagless" comes down to "was the JVM in which jtreg is running and about to launch a test, launched using certain set of JVM arguments? If it wasn't launched with those set of JVM arguments, then it is flagless." The set of such JVM arguments appears to be wide and may change over time.
@jaikiran "flagless" should mean that jtreg was not launched with `-javaoption` or `-vmoption` being set. But I see now (thanks for the link) it does allow for some flags to be set `<sigh>`
-------------
PR Comment: https://git.openjdk.org/jdk/pull/16572#issuecomment-1803302532
More information about the core-libs-dev
mailing list