arduino
diff --git a/Collapse file
‎buildcache/build_cache.go‎
Copy file name to clipboardExpand all lines: buildcache/build_cache.go
+38-7Lines changed: 38 additions & 7 deletions b/Collapse file
‎buildcache/build_cache.go‎
Copy file name to clipboardExpand all lines: buildcache/build_cache.go
+38-7Lines changed: 38 additions & 7 deletions
diff --git a/Collapse file
‎buildcache/build_cache_test.go‎
Copy file name to clipboardExpand all lines: buildcache/build_cache_test.go
+1-1Lines changed: 1 addition & 1 deletion b/Collapse file
‎buildcache/build_cache_test.go‎
Copy file name to clipboardExpand all lines: buildcache/build_cache_test.go
+1-1Lines changed: 1 addition & 1 deletion
diff --git a/Collapse file
‎buildcache/directory_cache.go‎
Copy file name to clipboardExpand all lines: buildcache/directory_cache.go
-100Lines changed: 0 additions & 100 deletions b/Collapse file
‎buildcache/directory_cache.go‎
Copy file name to clipboardExpand all lines: buildcache/directory_cache.go
-100Lines changed: 0 additions & 100 deletions
diff --git a/Collapse file
‎commands/compile/compile.go‎
Copy file name to clipboardExpand all lines: commands/compile/compile.go
+1-1Lines changed: 1 addition & 1 deletion b/Collapse file
‎commands/compile/compile.go‎
Copy file name to clipboardExpand all lines: commands/compile/compile.go
+1-1Lines changed: 1 addition & 1 deletion
diff --git a/Collapse file
‎legacy/builder/phases/core_builder.go‎
Copy file name to clipboardExpand all lines: legacy/builder/phases/core_builder.go
+1-1Lines changed: 1 addition & 1 deletion b/Collapse file
‎legacy/builder/phases/core_builder.go‎
Copy file name to clipboardExpand all lines: legacy/builder/phases/core_builder.go
+1-1Lines changed: 1 addition & 1 deletion
@@ -19,20 +19,51 @@ import (
 	"time"
 
 	"github.com/arduino/go-paths-helper"
+	"github.com/pkg/errors"
+	"github.com/sirupsen/logrus"
 )
 
-// GetOrCreate retrieves or creates the cache directory for the given key inside basePath.
+const lastUsedFileName = ".last-used"
+
+// GetOrCreate retrieves or creates the cache directory at the given path
 // If the cache already exists the lifetime of the cache is extended.
-func GetOrCreate(dir *paths.Path, key string) error {
-	unusedTTL := time.Hour
-	_, err := newDirectoryCache(dir, unusedTTL).
-		GetOrCreate(key, time.Now())
-	return err
+func GetOrCreate(dir *paths.Path) error {
+	if !dir.Exist() {
+		if err := dir.MkdirAll(); err != nil {
+			return err
+		}
+	}
+
+	return dir.Join(lastUsedFileName).WriteFile([]byte{})
 }
 
 // Purge removes all cache directories within baseDir that have expired
 // To know how long ago a directory has been last used
 // it checks into the .last-used file.
 func Purge(baseDir *paths.Path, ttl time.Duration) {
-	newDirectoryCache(baseDir, ttl).Purge()
+	files, err := baseDir.ReadDir()
+	if err != nil {
+		return
+	}
+	for _, file := range files {
+		if file.IsDir() {
+			removeIfExpired(file, ttl)
+		}
+	}
+}
+
+func removeIfExpired(dir *paths.Path, ttl time.Duration) {
+	fileInfo, err := dir.Join().Stat()
+	if err != nil {
+		return
+	}
+	lifeExpectancy := ttl - time.Since(fileInfo.ModTime())
+	if lifeExpectancy > 0 {
+		return
+	}
+	logrus.Tracef(`Purging cache directory "%s". Expired by %s`, dir, lifeExpectancy.Abs())
+	err = dir.RemoveAll()
+	if err != nil {
+		logrus.Tracef(`Error while pruning cache directory "%s": %s`, dir, errors.WithStack(err))
+	}
 }
@@ -43,7 +43,7 @@ func Test_UpdateLastUsedFileExisting(t *testing.T) {
 }
 
 func requireCorrectUpdate(t *testing.T, dir *paths.Path, prevModTime time.Time) {
-	require.NoError(t, GetOrCreate(dir.Parent(), dir.Base()))
+	require.NoError(t, GetOrCreate(dir))
 	expectedFile := dir.Join(lastUsedFileName)
 	fileInfo, err := expectedFile.Stat()
 	require.Nil(t, err)
 
@@ -138,7 +138,7 @@ func Compile(ctx context.Context, req *rpc.CompileRequest, outStream, errStream
 		return nil, &arduino.PermissionDeniedError{Message: tr("Cannot create build directory"), Cause: err}
 	}
 
-	buildcache.GetOrCreate(builderCtx.BuildPath.Parent(), builderCtx.BuildPath.Base())
+	buildcache.GetOrCreate(builderCtx.BuildPath)
 	// cache is purged after compilation to not remove entries that might be required
 	defer maybePurgeBuildCache()
 
 
@@ -95,7 +95,7 @@ func compileCore(ctx *types.Context, buildPath *paths.Path, buildCachePath *path
 	if buildCachePath != nil {
 		archivedCoreName := GetCachedCoreArchiveDirName(buildProperties.Get(constants.BUILD_PROPERTIES_FQBN),
 			buildProperties.Get("compiler.optimization_flags"), realCoreFolder)
-		buildcache.GetOrCreate(buildCachePath, archivedCoreName)
+		buildcache.GetOrCreate(buildCachePath.Join(archivedCoreName))
 		targetArchivedCore = buildCachePath.Join(archivedCoreName, "core.a")
 		canUseArchivedCore := !ctx.OnlyUpdateCompilationDatabase &&
 			!ctx.Clean &&
Original file line number	Diff line number	Diff line change
`@@ -43,7 +43,7 @@ func Test_UpdateLastUsedFileExisting(t *testing.T) {`
`43`	`43`	`}`
`44`	`44`
`45`	`45`	`func requireCorrectUpdate(t testing.T, dir paths.Path, prevModTime time.Time) {`
`46`		`- require.NoError(t, GetOrCreate(dir.Parent(), dir.Base()))`
	`46`	`+ require.NoError(t, GetOrCreate(dir))`
`47`	`47`	`expectedFile := dir.Join(lastUsedFileName)`
`48`	`48`	`fileInfo, err := expectedFile.Stat()`
`49`	`49`	`require.Nil(t, err)`
Original file line number	Diff line number	Diff line change
`@@ -138,7 +138,7 @@ func Compile(ctx context.Context, req *rpc.CompileRequest, outStream, errStream`
`138`	`138`	`return nil, &arduino.PermissionDeniedError{Message: tr("Cannot create build directory"), Cause: err}`
`139`	`139`	`}`
`140`	`140`
`141`		`- buildcache.GetOrCreate(builderCtx.BuildPath.Parent(), builderCtx.BuildPath.Base())`
	`141`	`+ buildcache.GetOrCreate(builderCtx.BuildPath)`
`142`	`142`	`// cache is purged after compilation to not remove entries that might be required`
`143`	`143`	`defer maybePurgeBuildCache()`
`144`	`144`