[whimsy] branch master updated: Ensure vars are defined

[se...@apache.org]

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 a89b9a2  Ensure vars are defined
a89b9a2 is described below

commit a89b9a24e0467b69d826c79ce4685f6e99077868
Author: Sebb <se...@apache.org>
AuthorDate: Wed Jun 17 12:02:35 2020 +0100

    Ensure vars are defined
---
 lib/whimsy/asf/svn.rb | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/lib/whimsy/asf/svn.rb b/lib/whimsy/asf/svn.rb
index 8f49bd6..a0bf5ac 100644
--- a/lib/whimsy/asf/svn.rb
+++ b/lib/whimsy/asf/svn.rb
@@ -660,13 +660,14 @@ module ASF
           syscmd << '--revision'
           syscmd << revision 
         end
+
+        sysopts = {}
         if env
           if self.passwordStdinOK?()
             syscmd << ['--username', env.user, '--password-from-stdin']
-            sysopts[:stdin] = password
+            sysopts[:stdin] = env.password
           else
             syscmd << ['--username', env.user, '--password', env.password]
-            sysopts = {}
           end
         end
         if _.instance_of?(Wunderbar::JsonBuilder) or _.instance_of?(Wunderbar::TextBuilder)