Skip to content
This repository has been archived by the owner on Dec 2, 2020. It is now read-only.

Commit

Permalink
nodebuild: s3_cellar is unnecessary
Browse files Browse the repository at this point in the history
This isn't needed as there are no references to Homebrew's Cellar (unlike Ruby): https://github.com/boxen/our-boxen/pull/739/files#r32968835.
  • Loading branch information
MikeMcQuaid committed Aug 20, 2015
1 parent a7291f7 commit dab3a5b
Showing 1 changed file with 0 additions and 12 deletions.
12 changes: 0 additions & 12 deletions lib/puppet/provider/nodejs/nodebuild.rb
Original file line number Diff line number Diff line change
Expand Up @@ -70,17 +70,6 @@ def tmp
"/tmp/node-#{version}.tar.bz2"
end

# Keep in sync with same-named function in:
# https://github.com/boxen/our-boxen/blob/master/script/sync
def s3_cellar
homebrew_cellar = "#{Facter.value(:homebrew_root)}/Cellar"
case homebrew_cellar
when "/Cellar", "/opt/boxen/homebrew/Cellar" then ""
when "/usr/local/Cellar" then "default/"
else "#{Base64.strict_encode64(homebrew_cellar)}/"
end
end

def precompiled_url
base = Facter.value(:boxen_download_url_base) ||
"https://#{Facter.value(:boxen_s3_host)}/#{Facter.value(:boxen_s3_bucket)}"
Expand All @@ -92,7 +81,6 @@ def precompiled_url
/
#{Facter.value(:operatingsystem)}
/
#{s3_cellar}
#{os_release}
/
#{CGI.escape(version)}.tar.bz2
Expand Down

0 comments on commit dab3a5b

Please sign in to comment.