Enfore using bash.exe on Windows

Author: koppor

CHANGELOG.md

@@ -14,6 +14,10 @@
 
 * Add `--key_file` to allow using a service account key file
 
+### pages
+
+* Fix to allow being run on a Windows Git bash
+
 ### Releases
 
 * Make --file glob files by default, and default to `*`

Gemfile

@@ -12,6 +12,8 @@ gems.each do |name, version, opts|
   gem name, version, opts
 end
 
+gem 'os'
+
 group :test do
   gem 'coveralls'
   gem 'rspec'

lib/dpl/ctx/bash.rb

@@ -2,6 +2,7 @@
 require 'fileutils'
 require 'logger'
 require 'open3'
+require 'os'
 require 'tmpdir'
 require 'securerandom'
 require 'dpl/support/version'
@@ -262,7 +263,17 @@ def shell(cmd, opts = {})
         info cmd.echo if cmd.echo?
 
         @last_out, @last_err, @last_status = retrying(cmd.retry ? 2 : 0) do
-          send(cmd.capture? ? :open3 : :system, cmd.cmd, cmd.opts)
+          if OS.windows?
+            command = "set -o pipefail && #{cmd.cmd}"
+            # required to get rsync working as rsync otherwise outputs "ssh: Could not resolve hostname c: Name or service not known"
+            command = command.sub("C:/", "/c/")
+            command = "bash -c \"#{command}\""
+            system("#{command}")
+            cmd_status = $?
+            return cmd_status.exitstatus == 0
+          else
+            send(cmd.capture? ? :open3 : :system, cmd.cmd, cmd.opts)
+          end
         end
 
         info cmd.success % { out: last_out } if success? && cmd.success?
@@ -405,14 +416,14 @@ def git_author_name
         `git log #{git_sha} -n 1 --pretty=%an`.chomp
       end
 
-      # Returns the comitter email of the commit `git_sha`.
+      # Returns the committer email of the commit `git_sha`.
       def git_author_email
         `git log #{git_sha} -n 1 --pretty=%ae`.chomp
       end
 
       # Whether or not the git working directory is dirty
       def git_dirty?
-        !Kernel.system('git diff --quiet')
+        !shell('git diff --quiet')
       end
 
       # Returns the output of `git log`, using the given args.
@@ -430,7 +441,7 @@ def git_ls_files
 
       # Returns true if the given ref exists remotely
       def git_ls_remote?(url, ref)
-        Kernel.system("git ls-remote --exit-code #{url} #{ref} > /dev/null 2>&1")
+        shell("git ls-remote --exit-code #{url} #{ref} > /dev/null 2>&1")
       end
 
       # Returns known Git remote URLs

lib/dpl/providers/pages/git.rb

@@ -96,7 +96,7 @@ def deploy
           info :committer_from_gh if committer_from_gh?
           info :git_config
           copy_files
-          return info :stop unless git_dirty?
+          return info :stop unless !(git_clone?) || git_dirty?
           git_config
           git_commit
           git_push