Fix redir version fetching.

"" || "something" doesn't really work. Have to check for the actual string length to determine if got something or not out of the pipe.
fgrehm · Sep 25, 2018 · f750bf4 · f750bf4
1 parent 2a5510b
commit f750bf4
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/lib/vagrant-lxc/action/forward_ports.rb b/lib/vagrant-lxc/action/forward_ports.rb
@@ -108,7 +108,7 @@ def store_redir_pid(host_port, redir_pid)
         def redir_version
           stdout, stderr, _ = Open3.capture3 "redir --version"
           # For some weird reason redir printed version information in STDERR prior to 3.2
-          version = stdout || stderr
+          version = stdout.empty? ? stderr : stdout
           version.split('.')[0].to_i
         end