Skip to content

Backport v6.x - build: use $(RM) in Makefile for consistency

Rodrigo Muino Tomonari requested to merge github/fork/gibfahn/rm-makefile-v6 into v6.x-staging

Backport-Of: https://github.com/nodejs/node/pull/12157

Also allows someone to reassign $RM, e.g. with RM=rm -v instead of rm -f (the default) should they want to. We're currently using a mixture of $(RM) and rm -f.

There are a couple of places which aren't doing -f, have them do it for consistency.

PR-URL: https://github.com/nodejs/node/pull/12157 Reviewed-By: Joyee Cheung joyeec9h3@gmail.com Reviewed-By: Richard Lau riclau@uk.ibm.com Reviewed-By: Anna Henningsen anna@addaleax.net Reviewed-By: Ben Noordhuis info@bnoordhuis.nl Reviewed-By: James M Snell jasnell@gmail.com Reviewed-By: Michael Dawson michael_dawson@ca.ibm.com Reviewed-By: Johan Bergström bugs@bergstroem.nu

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines
Affected core subsystem(s)

Merge request reports