Merge pull request #2294 from reitermarkus/cask-chmod-binary

Always `chmod +x` `binary` stanzas.

2 files changed, 6 insertions, 1 deletions

diff --git a/Library/Homebrew/cask/lib/hbc/artifact/binary.rb b/Library/Homebrew/cask/lib/hbc/artifact/binary.rb
index 9136d6a2a..395ab5c5c 100644
--- a/Library/Homebrew/cask/lib/hbc/artifact/binary.rb
+++ b/Library/Homebrew/cask/lib/hbc/artifact/binary.rb
@@ -6,6 +6,11 @@ module Hbc
       def install_phase
         super if CLI.binaries?
       end
+
+      def link(artifact_spec)
+        super(artifact_spec)
+        FileUtils.chmod "+x", source
+      end
     end
   end
 end
diff --git a/Library/Homebrew/cask/lib/hbc/artifact/relocated.rb b/Library/Homebrew/cask/lib/hbc/artifact/relocated.rb
index aa12dfa4a..bd68ccad0 100644
--- a/Library/Homebrew/cask/lib/hbc/artifact/relocated.rb
+++ b/Library/Homebrew/cask/lib/hbc/artifact/relocated.rb
@@ -35,7 +35,7 @@ module Hbc
         altnames = "(#{altnames})"
 
         # Some packges are shipped as u=rx (e.g. Bitcoin Core)
-        @command.run!("/bin/chmod", args: ["--", "u=rwx", file.to_s, file.realpath.to_s])
+        @command.run!("/bin/chmod", args: ["--", "u+rw", file.to_s, file.realpath.to_s])
 
         @command.run!("/usr/bin/xattr",
                       args:         ["-w", ALT_NAME_ATTRIBUTE, altnames, file.to_s],