legacy: Builder refactorization (part 1...)

legacy/builder/types/accessories.go → arduino/builder/builder.go

@@ -13,29 +13,18 @@
 // Arduino software without disclosing the source code of your own applications.
 // To purchase a commercial license, send an email to [email protected].
 
-package types
+package builder
 
-import "golang.org/x/exp/slices"
+import "github.com/arduino/arduino-cli/arduino/sketch"
 
-type UniqueSourceFileQueue []*SourceFile
-
-func (queue *UniqueSourceFileQueue) Push(value *SourceFile) {
-	if !queue.Contains(value) {
-		*queue = append(*queue, value)
-	}
-}
-
-func (queue UniqueSourceFileQueue) Contains(target *SourceFile) bool {
-	return slices.ContainsFunc(queue, target.Equals)
+// Builder is a Sketch builder.
+type Builder struct {
+	sketch *sketch.Sketch
 }
 
-func (queue *UniqueSourceFileQueue) Pop() *SourceFile {
-	old := *queue
-	x := old[0]
-	*queue = old[1:]
-	return x
-}
-
-func (queue UniqueSourceFileQueue) Empty() bool {
-	return len(queue) == 0
+// NewBuilder creates a sketch Builder.
+func NewBuilder(sk *sketch.Sketch) *Builder {
+	return &Builder{
+		sketch: sk,
+	}
 }

arduino/builder/compilation_database.go

@@ -19,8 +19,8 @@ import (
 	"encoding/json"
 	"fmt"
 	"os"
-	"os/exec"
 
+	"github.com/arduino/arduino-cli/executils"
 	"github.com/arduino/go-paths-helper"
 )
 
@@ -67,25 +67,22 @@ func (db *CompilationDatabase) SaveToFile() {
 	}
 }
 
-func dirForCommand(command *exec.Cmd) string {
-	// This mimics what Cmd.Run also does: Use Dir if specified,
-	// current directory otherwise
-	if command.Dir != "" {
-		return command.Dir
-	}
-	dir, err := os.Getwd()
-	if err != nil {
-		fmt.Println(tr("Error getting current directory for compilation database: %s", err))
-		return ""
+// Add adds a new CompilationDatabase entry
+func (db *CompilationDatabase) Add(target *paths.Path, command *executils.Process) {
+	commandDir := command.GetDir()
+	if commandDir == "" {
+		// This mimics what Cmd.Run also does: Use Dir if specified,
+		// current directory otherwise
+		dir, err := os.Getwd()
+		if err != nil {
+			fmt.Println(tr("Error getting current directory for compilation database: %s", err))
+		}
+		commandDir = dir
 	}
-	return dir
-}
 
-// Add adds a new CompilationDatabase entry
-func (db *CompilationDatabase) Add(target *paths.Path, command *exec.Cmd) {
 	entry := CompilationCommand{
-		Directory: dirForCommand(command),
-		Arguments: command.Args,
+		Directory: commandDir,
+		Arguments: command.GetArgs(),
 		File:      target.String(),
 	}
 

arduino/builder/compilation_database_test.go

@@ -16,9 +16,9 @@
 package builder
 
 import (
-	"os/exec"
 	"testing"
 
+	"github.com/arduino/arduino-cli/executils"
 	"github.com/arduino/go-paths-helper"
 	"github.com/stretchr/testify/require"
 )
@@ -28,7 +28,8 @@ func TestCompilationDatabase(t *testing.T) {
 	require.NoError(t, err)
 	defer tmpfile.Remove()
 
-	cmd := exec.Command("gcc", "arg1", "arg2")
+	cmd, err := executils.NewProcess(nil, "gcc", "arg1", "arg2")
+	require.NoError(t, err)
 	db := NewCompilationDatabase(tmpfile)
 	db.Add(paths.New("test"), cmd)
 	db.SaveToFile()

arduino/builder/cpp/cpp.go

@@ -106,3 +106,8 @@ func ParseString(line string) (string, string, bool) {
 		i += width
 	}
 }
+
+// WrapWithHyphenI fixdoc
+func WrapWithHyphenI(value string) string {
+	return "\"-I" + value + "\""
+}