Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Improved compile speed by running multi-threaded library discovery. #2625

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 18 commits into from
Jul 25, 2025
Merged

Improved compile speed by running multi-threaded library discovery. #2625

Changes from 1 commit
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
97451d8
Simplified error reporting in library detection
cmaglie May 31, 2024
9b7b9d2
Remove useless targetFilePath variable
cmaglie Jun 1, 2024
a75945a
Slight improvement of removeBuildFromSketchFiles
cmaglie Jun 2, 2024
c00b637
Rename variables for clarity
cmaglie Jun 2, 2024
ddd1e5b
Removed hardcoded build.warn_data_percentage in build.options file
cmaglie Jun 2, 2024
688f28f
Renamed variables for clarity
cmaglie Jun 2, 2024
7eb04d0
Renamed variables for clarity
cmaglie Jun 2, 2024
ba8b764
Pre-compute sourceFile fields, and save the in the includes.cache
cmaglie Jun 3, 2024
eee25d1
Added ObjFileIsUpToDate method to sourceFile
cmaglie Jun 3, 2024
6247a03
Implemented parallel task runner
cmaglie Jun 4, 2024
8013f91
Simplify use of properties.SplitQuotedString
cmaglie Jun 4, 2024
142c297
Use runner.Task in GCC preprocessor
cmaglie Jun 5, 2024
14a3adb
Parallelize library discovery phase in compile
cmaglie Jun 5, 2024
b8f94f5
The number of jobs in library detection now follows --jobs flag
cmaglie Jun 11, 2024
d0d4173
Reordered properties construction for clarity
cmaglie Jun 13, 2024
0579d02
Reordered compileFileWithRecipe for clarity
cmaglie Jun 13, 2024
b717a2a
Added integration test
cmaglie Jun 12, 2024
7241f25
fix: libraries are recompiled if the list of include paths changes
cmaglie Jun 13, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Slight improvement of removeBuildFromSketchFiles
  • Loading branch information
@cmaglie
cmaglie committed Jul 14, 2025
commit a75945a292cc29c2a2520c3f68a1a1b4952ce8cb
10 changes: 5 additions & 5 deletions commands/service_compile.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -169,7 +169,7 @@ func (s *arduinoCoreServerImpl) Compile(req *rpc.CompileRequest, stream rpc.Ardu
if buildPathArg := req.GetBuildPath(); buildPathArg != "" {
buildPath = paths.New(req.GetBuildPath()).Canonical()
if in, _ := buildPath.IsInsideDir(sk.FullPath); in && buildPath.IsDir() {
if sk.AdditionalFiles, err = removeBuildFromSketchFiles(sk.AdditionalFiles, buildPath); err != nil {
if sk.AdditionalFiles, err = removeBuildPathFromSketchFiles(sk.AdditionalFiles, buildPath); err != nil {
return err
}
}
Expand Down Expand Up @@ -462,15 +462,15 @@ func maybePurgeBuildCache(compilationsBeforePurge uint, cacheTTL time.Duration)
buildcache.New(paths.TempDir().Join("arduino", "sketches")).Purge(cacheTTL)
}

// removeBuildFromSketchFiles removes the files contained in the build directory from
// removeBuildPathFromSketchFiles removes the files contained in the build directory from
// the list of the sketch files
func removeBuildFromSketchFiles(files paths.PathList, build *paths.Path) (paths.PathList, error) {
func removeBuildPathFromSketchFiles(files paths.PathList, build *paths.Path) (paths.PathList, error) {
var res paths.PathList
ignored := false
for _, file := range files {
if isInside, _ := file.IsInsideDir(build); !isInside {
res = append(res, file)
} else if !ignored {
res.Add(file)
} else {
ignored = true
}
}
Expand Down