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