diff options
| author | Vítor Galvão | 2017-01-28 16:25:14 +0000 |
|---|---|---|
| committer | Vítor Galvão | 2017-01-28 18:29:59 +0000 |
| commit | c0b84a0479562edd0bb56b3d915fda9f90a520da (patch) | |
| tree | 14f281cf98b5cf3d7079c117f6b01eef04ffe2e5 | |
| parent | ad4fd014e1e558207d4eecdafe8667ecba5a8f51 (diff) | |
| download | brew-c0b84a0479562edd0bb56b3d915fda9f90a520da.tar.bz2 | |
cask: staged: no sudo in set_permissions
| -rw-r--r-- | Library/Homebrew/cask/lib/hbc/staged.rb | 2 | ||||
| -rw-r--r-- | Library/Homebrew/cask/test/support/shared_examples/staged.rb | 4 |
2 files changed, 3 insertions, 3 deletions
diff --git a/Library/Homebrew/cask/lib/hbc/staged.rb b/Library/Homebrew/cask/lib/hbc/staged.rb index dcc05ef5d..c1aa01b29 100644 --- a/Library/Homebrew/cask/lib/hbc/staged.rb +++ b/Library/Homebrew/cask/lib/hbc/staged.rb @@ -27,7 +27,7 @@ module Hbc full_paths = remove_nonexistent(paths) return if full_paths.empty? @command.run!("/bin/chmod", args: ["-R", "--", permissions_str] + full_paths, - sudo: true) + sudo: false) end def set_ownership(paths, user: current_user, group: "staff") diff --git a/Library/Homebrew/cask/test/support/shared_examples/staged.rb b/Library/Homebrew/cask/test/support/shared_examples/staged.rb index ef4ff0341..95e75726e 100644 --- a/Library/Homebrew/cask/test/support/shared_examples/staged.rb +++ b/Library/Homebrew/cask/test/support/shared_examples/staged.rb @@ -47,7 +47,7 @@ shared_examples_for Hbc::Staged do staged.stubs(Pathname: fake_pathname) Hbc::FakeSystemCommand.expects_command( - ["/usr/bin/sudo", "-E", "--", "/bin/chmod", "-R", "--", "777", fake_pathname] + ["/bin/chmod", "-R", "--", "777", fake_pathname] ) staged.set_permissions(fake_pathname.to_s, "777") end @@ -57,7 +57,7 @@ shared_examples_for Hbc::Staged do staged.stubs(:Pathname).returns(fake_pathname) Hbc::FakeSystemCommand.expects_command( - ["/usr/bin/sudo", "-E", "--", "/bin/chmod", "-R", "--", "777", fake_pathname, fake_pathname] + ["/bin/chmod", "-R", "--", "777", fake_pathname, fake_pathname] ) staged.set_permissions([fake_pathname.to_s, fake_pathname.to_s], "777") end |