Merge pull request #967 from heroku/schneems/quote-export-file

[changelog skip] Quote buildpack's `export` values

lib/language_pack/base.rb

@@ -253,9 +253,9 @@ def export(key, val, layer: nil, option: nil)
     else
       string =
         if option == :default
-          "export #{key}=${#{key}:-#{val}}"
+          %{export #{key}="${#{key}:-#{val}}"}
         elsif option == :path
-          "export #{key}=#{val}:$#{key}"
+          %{export #{key}="#{val}:$#{key}"}
         else
           %{export #{key}="#{val.gsub('"','\"')}"}
         end

spec/hatchet/buildpack_spec.rb

@@ -0,0 +1,15 @@
+require_relative '../spec_helper'
+
+describe "Buildpack internals" do
+  it "handles PATH with a newline in it correctly" do
+    buildpacks = [
+      "https://github.com/sharpstone/export_path_with_newlines_buildpack",
+      Hatchet::App.default_buildpack,
+      "https://github.com/heroku/null-buildpack"
+    ]
+    Hatchet::Runner.new("default_ruby", buildpacks: buildpacks).deploy do |app|
+      expect(app.output).to_not match("No such file or directory")
+    end
+  end
+end
+