move coreBuildCachePath in the arduino Builder

Author: alessio-perugini

arduino/builder/builder.go

@@ -15,7 +15,10 @@
 
 package builder
 
-import "github.com/arduino/arduino-cli/arduino/sketch"
+import (
+	"github.com/arduino/arduino-cli/arduino/sketch"
+	"github.com/arduino/go-paths-helper"
+)
 
 // nolint
 const (
@@ -31,11 +34,15 @@ const (
 // Builder is a Sketch builder.
 type Builder struct {
 	sketch *sketch.Sketch
+
+	// core related
+	coreBuildCachePath *paths.Path
 }
 
 // NewBuilder creates a sketch Builder.
-func NewBuilder(sk *sketch.Sketch) *Builder {
+func NewBuilder(sk *sketch.Sketch, coreBuildCachePath *paths.Path) *Builder {
 	return &Builder{
-		sketch: sk,
+		sketch:             sk,
+		coreBuildCachePath: coreBuildCachePath,
 	}
 }

arduino/builder/core.go

@@ -0,0 +1,8 @@
+package builder
+
+import "github.com/arduino/go-paths-helper"
+
+// CoreBuildCachePath fixdoc
+func (b *Builder) CoreBuildCachePath() *paths.Path {
+	return b.coreBuildCachePath
+}

arduino/builder/sketch_test.go

@@ -48,7 +48,7 @@ func TestMergeSketchSources(t *testing.T) {
 	}
 	mergedSources := strings.ReplaceAll(string(mergedBytes), "%s", pathToGoldenSource)
 
-	b := NewBuilder(sk)
+	b := NewBuilder(sk, nil)
 	offset, source, err := b.sketchMergeSources(nil)
 	require.Nil(t, err)
 	require.Equal(t, 2, offset)
@@ -61,7 +61,7 @@ func TestMergeSketchSourcesArduinoIncluded(t *testing.T) {
 	require.NotNil(t, sk)
 
 	// ensure not to include Arduino.h when it's already there
-	b := NewBuilder(sk)
+	b := NewBuilder(sk, nil)
 	_, source, err := b.sketchMergeSources(nil)
 	require.Nil(t, err)
 	require.Equal(t, 1, strings.Count(source, "<Arduino.h>"))
@@ -76,7 +76,7 @@ func TestCopyAdditionalFiles(t *testing.T) {
 	sk1, err := sketch.New(paths.New("testdata", t.Name()))
 	require.Nil(t, err)
 	require.Equal(t, sk1.AdditionalFiles.Len(), 1)
-	b1 := NewBuilder(sk1)
+	b1 := NewBuilder(sk1, nil)
 
 	// copy the sketch over, create a fake main file we don't care about it
 	// but we need it for `SketchLoad` to succeed later

commands/compile/compile.go

@@ -155,7 +155,21 @@ func Compile(ctx context.Context, req *rpc.CompileRequest, outStream, errStream
 	// cache is purged after compilation to not remove entries that might be required
 	defer maybePurgeBuildCache()
 
-	sketchBuilder := bldr.NewBuilder(sk)
+	var coreBuildCachePath *paths.Path
+	if req.GetBuildCachePath() == "" {
+		coreBuildCachePath = paths.TempDir().Join("arduino", "cores")
+	} else {
+		buildCachePath, err := paths.New(req.GetBuildCachePath()).Abs()
+		if err != nil {
+			return nil, &arduino.PermissionDeniedError{Message: tr("Cannot create build cache directory"), Cause: err}
+		}
+		if err := buildCachePath.MkdirAll(); err != nil {
+			return nil, &arduino.PermissionDeniedError{Message: tr("Cannot create build cache directory"), Cause: err}
+		}
+		coreBuildCachePath = buildCachePath.Join("core")
+	}
+
+	sketchBuilder := bldr.NewBuilder(sk, coreBuildCachePath)
 
 	// Add build properites related to sketch data
 	buildProperties = sketchBuilder.SetupBuildProperties(buildProperties, buildPath, req.GetOptimizeForDebug())
@@ -206,19 +220,6 @@ func Compile(ctx context.Context, req *rpc.CompileRequest, outStream, errStream
 		builderCtx.WarningsLevel = builder.DEFAULT_WARNINGS_LEVEL
 	}
 
-	if req.GetBuildCachePath() == "" {
-		builderCtx.CoreBuildCachePath = paths.TempDir().Join("arduino", "cores")
-	} else {
-		buildCachePath, err := paths.New(req.GetBuildCachePath()).Abs()
-		if err != nil {
-			return nil, &arduino.PermissionDeniedError{Message: tr("Cannot create build cache directory"), Cause: err}
-		}
-		if err := buildCachePath.MkdirAll(); err != nil {
-			return nil, &arduino.PermissionDeniedError{Message: tr("Cannot create build cache directory"), Cause: err}
-		}
-		builderCtx.CoreBuildCachePath = buildCachePath.Join("core")
-	}
-
 	builderCtx.BuiltInLibrariesDirs = configuration.IDEBuiltinLibrariesDir(configuration.Settings)
 
 	builderCtx.Stdout = outStream

legacy/builder/builder.go

@@ -73,8 +73,8 @@ func (s *Builder) Run(ctx *types.Context) error {
 		&RecipeByPrefixSuffixRunner{Prefix: "recipe.hooks.core.prebuild", Suffix: ".pattern"},
 
 		types.BareCommand(func(ctx *types.Context) error {
-			objectFiles, archiveFile, coreBuildCachePath, err := phases.CoreBuilder(
-				ctx.BuildPath, ctx.CoreBuildPath, ctx.CoreBuildCachePath,
+			objectFiles, archiveFile, err := phases.CoreBuilder(
+				ctx.BuildPath, ctx.CoreBuildPath, ctx.Builder.CoreBuildCachePath(),
 				ctx.BuildProperties,
 				ctx.ActualPlatform,
 				ctx.Verbose, ctx.OnlyUpdateCompilationDatabase, ctx.Clean,
@@ -90,7 +90,6 @@ func (s *Builder) Run(ctx *types.Context) error {
 
 			ctx.CoreObjectsFiles = objectFiles
 			ctx.CoreArchiveFilePath = archiveFile
-			ctx.CoreBuildCachePath = coreBuildCachePath
 
 			return err
 		}),

legacy/builder/phases/core_builder.go

@@ -52,9 +52,9 @@ func CoreBuilder(
 	verboseInfoFn func(msg string),
 	verboseStdoutFn, verboseStderrFn func(data []byte),
 	progress *progress.Struct, progressCB rpc.TaskProgressCB,
-) (paths.PathList, *paths.Path, *paths.Path, error) {
+) (paths.PathList, *paths.Path, error) {
 	if err := coreBuildPath.MkdirAll(); err != nil {
-		return nil, nil, coreBuildCachePath, errors.WithStack(err)
+		return nil, nil, errors.WithStack(err)
 	}
 
 	if coreBuildCachePath != nil {
@@ -63,7 +63,7 @@ func CoreBuilder(
 			verboseInfoFn(tr("Running normal build of the core..."))
 			coreBuildCachePath = nil
 		} else if err := coreBuildCachePath.MkdirAll(); err != nil {
-			return nil, nil, coreBuildCachePath, errors.WithStack(err)
+			return nil, nil, errors.WithStack(err)
 		}
 	}
 
@@ -81,10 +81,10 @@ func CoreBuilder(
 		progress, progressCB,
 	)
 	if err != nil {
-		return nil, nil, coreBuildCachePath, errors.WithStack(err)
+		return nil, nil, errors.WithStack(err)
 	}
 
-	return objectFiles, archiveFile, coreBuildCachePath, nil
+	return objectFiles, archiveFile, nil
 }
 
 func compileCore(

legacy/builder/test/builder_test.go

@@ -97,7 +97,7 @@ func prepareBuilderTestContext(t *testing.T, ctx *types.Context, sketchPath *pat
 		ctx.Sketch = sk
 	}
 
-	ctx.Builder = bldr.NewBuilder(ctx.Sketch)
+	ctx.Builder = bldr.NewBuilder(ctx.Sketch, nil)
 	if fqbn != "" {
 		ctx.FQBN = parseFQBN(t, fqbn)
 		targetPackage, targetPlatform, targetBoard, buildProperties, buildPlatform, err := pme.ResolveFQBN(ctx.FQBN)

legacy/builder/test/helper.go

@@ -21,7 +21,6 @@ import (
 
 	"github.com/arduino/arduino-cli/arduino/cores"
 	"github.com/arduino/arduino-cli/arduino/libraries"
-	"github.com/arduino/arduino-cli/legacy/builder/constants"
 	"github.com/arduino/arduino-cli/legacy/builder/types"
 	paths "github.com/arduino/go-paths-helper"
 	"github.com/stretchr/testify/require"
@@ -40,13 +39,6 @@ func SetupBuildPath(t *testing.T, ctx *types.Context) *paths.Path {
 	return buildPath
 }
 
-func SetupBuildCachePath(t *testing.T, ctx *types.Context) *paths.Path {
-	buildCachePath, err := paths.MkTempDir(constants.EMPTY_STRING, "test_build_cache")
-	require.NoError(t, err)
-	ctx.CoreBuildCachePath = buildCachePath
-	return buildCachePath
-}
-
 func parseFQBN(t *testing.T, fqbnIn string) *cores.FQBN {
 	fqbn, err := cores.ParseFQBN(fqbnIn)
 	require.NoError(t, err)

legacy/builder/types/context.go

@@ -61,7 +61,6 @@ type Context struct {
 	BuildPath            *paths.Path
 	SketchBuildPath      *paths.Path
 	CoreBuildPath        *paths.Path
-	CoreBuildCachePath   *paths.Path
 	CoreArchiveFilePath  *paths.Path
 	CoreObjectsFiles     paths.PathList
 	LibrariesBuildPath   *paths.Path