fix: "host" builder tar paths

Signed-off-by: Matej Vasek <[email protected]>
knative · Jul 13, 2023 · be42b7a · be42b7a
1 parent 8f0b7c2
commit be42b7a
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/pkg/oci/containerize.go b/pkg/oci/containerize.go
@@ -7,6 +7,7 @@ import (
 	"fmt"
 	"io"
 	"os"
+	slashpath "path"
 	"path/filepath"
 	"strings"
 
@@ -159,7 +160,7 @@ func newDataTarball(source, target string, ignored []string, verbose bool) error
 			return err
 		}
 
-		header.Name = filepath.Join("/func", relPath)
+		header.Name = slashpath.Join("/func", filepath.ToSlash(relPath))
 		// TODO: should we set file timestamps to the build start time of cfg.t?
 		// header.ModTime = timestampArgument
 

diff --git a/pkg/oci/containerize_go.go b/pkg/oci/containerize_go.go
@@ -7,6 +7,7 @@ import (
 	"io"
 	"os"
 	"os/exec"
+	slashpath "path"
 	"path/filepath"
 	"strings"
 
@@ -170,7 +171,7 @@ func newExecTarball(source, target string, verbose bool) error {
 		return err
 	}
 
-	header.Name = path("/func", "f")
+	header.Name = slashpath.Join("/func", "f")
 	// TODO: should we set file timestamps to the build start time of cfg.t?
 	// header.ModTime = timestampArgument