From dab3a5bd8b95874cd78cf87e48d9bdc4c9ebe6de Mon Sep 17 00:00:00 2001 From: Mike McQuaid Date: Thu, 20 Aug 2015 14:06:47 +0100 Subject: [PATCH] nodebuild: s3_cellar is unnecessary 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. --- lib/puppet/provider/nodejs/nodebuild.rb | 12 ------------ 1 file changed, 12 deletions(-) diff --git a/lib/puppet/provider/nodejs/nodebuild.rb b/lib/puppet/provider/nodejs/nodebuild.rb index 5d2f7b5..4baec77 100644 --- a/lib/puppet/provider/nodejs/nodebuild.rb +++ b/lib/puppet/provider/nodejs/nodebuild.rb @@ -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)}" @@ -92,7 +81,6 @@ def precompiled_url / #{Facter.value(:operatingsystem)} / - #{s3_cellar} #{os_release} / #{CGI.escape(version)}.tar.bz2