buildpacks / samples

Samples for Cloud Native Buildpacks
Apache License 2.0
194 stars 145 forks source link

Fix the command parameter to work with Buildpacks API 0.9 #163

Closed dlion closed 1 year ago

dlion commented 1 year ago

In the example we are using Buildpacks API 0.9 https://github.com/buildpacks/samples/blob/main/apps/bash-script/bash-script-buildpack/buildpack.toml#L2 which requires to have a command = [something] as specified in the spec: https://github.com/buildpacks/spec/blob/buildpack/v0.9/buildpack.md#launchtoml-toml But in the builder we have a string instead of an array, causing an error when processed: https://github.com/buildpacks/samples/blob/main/apps/bash-script/bash-script-buildpack/bin/build#L13

This PR just fix that putting the ./app.bat or ./app.sh command into an array