In compile command try to release the package manager lock ASAP (#2741)

cmaglie · web-flow · commit ca4a4ece90ac · 2024-11-21T18:41:31.000+01:00
* In `compile` command try to release the package manager lock ASAP

* Applied suggestion from code review
diff --git a/commands/service_compile.go b/commands/service_compile.go
@@ -22,6 +22,7 @@ import (
 	"io"
 	"sort"
 	"strings"
+	"sync"
 	"time"
 
 	"github.com/arduino/arduino-cli/commands/cmderrors"
@@ -81,6 +82,7 @@ func (s *arduinoCoreServerImpl) Compile(req *rpc.CompileRequest, stream rpc.Ardu
 	if err != nil {
 		return err
 	}
+	release = sync.OnceFunc(release)
 	defer release()
 
 	if pme.Dirty() {
@@ -358,6 +360,10 @@ func (s *arduinoCoreServerImpl) Compile(req *rpc.CompileRequest, stream rpc.Ardu
 				targetBoard.String(), "'build.board'", sketchBuilder.GetBuildProperties().Get("build.board")) + "\n"))
 	}
 
+	// Release package manager
+	release()
+
+	// Perform the actual build
 	if err := sketchBuilder.Build(); err != nil {
 		return &cmderrors.CompileFailedError{Message: err.Error()}
 	}
