[whimsy] branch master updated: Use _.system for dryrun output

[sebb]

This is an automated email from the ASF dual-hosted git repository.

sebb pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/whimsy.git


The following commit(s) were added to refs/heads/master by this push:
     new 5c1a1b7  Use _.system for dryrun output
5c1a1b7 is described below

commit 5c1a1b7272556c9999d86cc688b6278d868dd709
Author: Sebb <s...@apache.org>
AuthorDate: Sun Jun 7 20:32:56 2020 +0100

    Use _.system for dryrun output
---
 lib/whimsy/asf/svn.rb | 12 +++++-------
 1 file changed, 5 insertions(+), 7 deletions(-)

diff --git a/lib/whimsy/asf/svn.rb b/lib/whimsy/asf/svn.rb
index e85a2d7..b11cbba 100644
--- a/lib/whimsy/asf/svn.rb
+++ b/lib/whimsy/asf/svn.rb
@@ -366,7 +366,10 @@ module ASF
         end
       end
 
-      p cmd if options[:dryrun] # before creds added
+      if options[:dryrun] # before creds added
+        # TODO: improve this
+        return _.system ['echo', cmd.inspect]
+      end
 
       # add credentials if required
       open_opts = {}
@@ -392,12 +395,7 @@ module ASF
 
       p cmd if options[:verbose] # includes auth
 
-      # issue svn command
-      if options[:dryrun]
-        0 # TODO is this the correct return value?
-      else
-        _.system cmd
-      end
+      _.system cmd
     end
 
     # retrieve revision, [err] for a path in svn