somemore PR comments, have not moved execd_wapper to include-files yet

buildpacks · Jul 20, 2022 · 5fa7981 · 5fa7981
1 parent bfae153
commit 5fa7981
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/profile/build.go b/profile/build.go
@@ -54,7 +54,7 @@ func Build(context libcnb.BuildContext) (libcnb.BuildResult, error) {
 		return result, err
 	}
 
-	err = os.Chmod(execPath, 0111)
+	err = os.Chmod(execPath, 0755)
 	if err != nil {
 		return result, err
 	}

diff --git a/profile/build_test.go b/profile/build_test.go
@@ -69,7 +69,7 @@ func (b BuildTest) RemoveWorkspace() BuildTest {
 }
 
 func (b BuildTest) Build() (libcnb.BuildContext, ExpectFunc, AfterFunc) {
-	outputPath, _ := os.MkdirTemp("/tmp", "dotprofile_out")
+	outputPath, _ := os.MkdirTemp("", "dotprofile_out")
 	os.MkdirAll(outputPath, os.ModePerm)
 	b.context.Layers.Path = outputPath
 	return b.context, b.expect, func() {

diff --git a/profile/detect.go b/profile/detect.go
@@ -27,7 +27,7 @@ import (
 func Detect(context libcnb.DetectContext) (libcnb.DetectResult, error) {
 	// NOTE: the logger is not passed into this function, that will likely be a change in libcnbv2
 
-	_, shErr := exec.LookPath("sh")
+	_, shErr := exec.LookPath("bash")
 
 	profilePath := filepath.Join(context.ApplicationPath, scriptName)
 	if _, err := os.Stat(profilePath); shErr == nil && !os.IsNotExist(err) {