Merge pull request #179233 from Homebrew/q-rubocop-no-rmrf

q*: Fix `Homebrew/NoFileutilsRmrf` RuboCop offenses
Homebrew · Aug 1, 2024 · a1a81cd · a1a81cd
2 parents 829c4a9 + 8bb9a14
commit a1a81cd
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/Formula/q/[email protected] b/Formula/q/[email protected]
@@ -256,10 +256,10 @@ class QtAT5 < Formula
   end
 
   def install
-    (buildpath/"qtwebengine").rmtree
+    rm_r(buildpath/"qtwebengine")
     (buildpath/"qtwebengine").install resource("qtwebengine")
 
-    (buildpath/"qtwebengine/src/3rdparty/chromium/third_party/catapult").rmtree
+    rm_r(buildpath/"qtwebengine/src/3rdparty/chromium/third_party/catapult")
     (buildpath/"qtwebengine/src/3rdparty/chromium/third_party/catapult").install resource("catapult")
 
     # FIXME: GN requires clang in clangBasePath/bin

diff --git a/Formula/q/questdb.rb b/Formula/q/questdb.rb
@@ -23,7 +23,7 @@ class Questdb < Formula
   depends_on "openjdk"
 
   def install
-    rm_rf "questdb.exe"
+    rm_r("questdb.exe")
     libexec.install Dir["*"]
     (bin/"questdb").write_env_script libexec/"questdb.sh", Language::Java.overridable_java_home_env
     inreplace libexec/"questdb.sh", "/usr/local/var/questdb", var/"questdb"

diff --git a/Formula/q/quictls.rb b/Formula/q/quictls.rb
@@ -102,7 +102,7 @@ def quictlsdir
   end
 
   def post_install
-    rm_f quictlsdir/"cert.pem"
+    rm(quictlsdir/"cert.pem") if (quictlsdir/"cert.pem").exist?
     quictlsdir.install_symlink Formula["ca-certificates"].pkgetc/"cert.pem"
   end