arduino_ci icon indicating copy to clipboard operation
arduino_ci copied to clipboard

Arduino-cli 0.14 has deprecated '--dry-run' flag since it is default behaviour since this version

Open s-t-a-n opened this issue 3 years ago • 2 comments

The latest arduino-cli version (0.14) no longer supports the '--dry-run' flag since 'Arduino CLI now defaults to the --dry-run behavior, so there is no longer any need for that flag (source).

I am not that familiar with Ruby, but I have tested removing --dry-run from lib/arduino_ci/arduino_backend.rb and this works fine for 0.14. On 0.13 this does create build/ directory with binaries.

Add version sensing for arduino-cli to arduino_backend.rb or maybe remove '--dry-run' and add build directory to gitignore ?

s-t-a-n avatar Jan 28 '21 22:01 s-t-a-n

Hey, thanks for reporting this! I think this is more generally a reminder that I should be upgrading to 0.14 of the arduino-cli backend. Let me set up that list and add this to it.

note to self: https://github.com/arduino/arduino-cli/issues/1051

ianfixes avatar Jan 29 '21 03:01 ianfixes

I've also applied the following bodge patch to /var/lib/gems/2.7.0/gems/arduino_ci-1.3.0/lib/arduino_ci

--- arduino_backend.rb.orig     2022-09-28 22:09:14.031508917 +0100
+++ arduino_backend.rb  2022-09-28 22:09:29.471458900 +0100
@@ -163,7 +163,7 @@
         @last_msg = "Can't compile Sketch at nonexistent path '#{path}'!"
         return false
       end
-      ret = run_and_capture("compile", "--fqbn", boardname, "--warnings", "all", "--dry-run", path.to_s)
+      ret = run_and_capture("compile", "--fqbn", boardname, "--warnings", "all", path.to_s)
       ret[:success]
     end

arduino-cli is now on version 0.27.1 and well deprecating --dry-run so it would be great to have PR #323 merged and released.

matburnham avatar Sep 28 '22 21:09 matburnham