RFR: 7903989: Default to no error if existing, make parent directories as needed
Christian Stein
cstein at openjdk.org
Tue Apr 15 14:56:43 UTC 2025
On Tue, 15 Apr 2025 14:43:13 GMT, Christian Stein <cstein at openjdk.org> wrote:
>> test/jdkOptsTest/JDKOptsTest.gmk line 38:
>>
>>> 36: -Xlint -Werror \
>>> 37: -encoding ASCII $(TESTDIR)/jdkOptsTest/JDKOptsTest.java
>>> 38: $(RM) -rf $(@:%.ok=%)/tmp
>>
>> `$(RM)` is already defined to include the `-rf`, so suggest you remove those options here
>
> Good catch. I only searched for `$(RM) -f` patterns before. Searching for `$(RM) -rf` hit two locations, both addressed in https://github.com/openjdk/jtreg/pull/259/commits/167328701a7f253b0eadbfe638545063fb217e5e
By the way, no hits for `$(RM) -fr` ... 😉
-------------
PR Review Comment: https://git.openjdk.org/jtreg/pull/259#discussion_r2044804160
More information about the jtreg-dev
mailing list