Commit d7efac1

author

Luca Bianconi

committed

refactor: new interface

1 parent 3b11d67 commit d7efac1Copy full SHA for d7efac1

File tree

3 files changed

+22

-12

lines changed

buildcache
- build_cache.go
- build_cache_test.go
commands/compile
- compile.go

3 files changed

+22

-12

lines changed

`‎buildcache/build_cache.go‎`

Lines changed: 17 additions & 7 deletions

Original file line number	Diff line number	Diff line change
`@@ -25,26 +25,31 @@ import (`
`25`	`25`
`26`	`26`	`const lastUsedFileName = ".last-used"`
`27`	`27`
	`28`	`+type buildCache struct {`
	`29`	`+ baseDir *paths.Path`
	`30`	`+}`
	`31`	`+`
`28`	`32`	`// GetOrCreate retrieves or creates the cache directory at the given path`
`29`	`33`	`// If the cache already exists the lifetime of the cache is extended.`
`30`		`-func GetOrCreate(dir paths.Path) (paths.Path, error) {`
`31`		`- if !dir.Exist() {`
`32`		`- if err := dir.MkdirAll(); err != nil {`
	`34`	`+func (bc buildCache) GetOrCreate(key string) (paths.Path, error) {`
	`35`	`+ keyDir := bc.baseDir.Join(key)`
	`36`	`+ if !keyDir.Exist() {`
	`37`	`+ if err := keyDir.MkdirAll(); err != nil {`
`33`	`38`	`return nil, err`
`34`	`39`	`}`
`35`	`40`	`}`
`36`	`41`
`37`		`- if err := dir.Join(lastUsedFileName).WriteFile([]byte{}); err != nil {`
	`42`	`+ if err := keyDir.Join(lastUsedFileName).WriteFile([]byte{}); err != nil {`
`38`	`43`	`return nil, err`
`39`	`44`	`}`
`40`		`- return dir, nil`
	`45`	`+ return keyDir, nil`
`41`	`46`	`}`
`42`	`47`
`43`	`48`	`// Purge removes all cache directories within baseDir that have expired`
`44`	`49`	`// To know how long ago a directory has been last used`
`45`	`50`	`// it checks into the .last-used file.`
`46`		`-func Purge(baseDir*paths.Path, ttl time.Duration) {`
`47`		`- files, err := baseDir.ReadDir()`
	`51`	`+func (bc*buildCache) Purge(ttl time.Duration) {`
	`52`	`+ files, err := bc.baseDir.ReadDir()`
`48`	`53`	`if err != nil {`
`49`	`54`	`return`
`50`	`55`	`}`
`@@ -55,6 +60,11 @@ func Purge(baseDir *paths.Path, ttl time.Duration) {`
`55`	`60`	`}`
`56`	`61`	`}`
`57`	`62`
	`63`	`+// New instantiates a build cache`
	`64`	`+func New(baseDir paths.Path) buildCache {`
	`65`	`+ return &buildCache{baseDir}`
	`66`	`+}`
	`67`	`+`
`58`	`68`	`func removeIfExpired(dir *paths.Path, ttl time.Duration) {`
`59`	`69`	`fileInfo, err := dir.Join().Stat()`
`60`	`70`	`if err != nil {`

`‎buildcache/build_cache_test.go‎`

Lines changed: 2 additions & 2 deletions

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`		`- _, err := GetOrCreate(dir)`
	`46`	`+ _, err := New(dir.Parent()).GetOrCreate(dir.Base())`
`47`	`47`	`require.NoError(t, err)`
`48`	`48`	`expectedFile := dir.Join(lastUsedFileName)`
`49`	`49`	`fileInfo, err := expectedFile.Stat()`
`@@ -71,7 +71,7 @@ func TestPurge(t *testing.T) {`
`71`	`71`	`require.NoError(t, infoFilePath.Chtimes(accesstime, lastUsedTime))`
`72`	`72`	`}`
`73`	`73`
`74`		`- Purge(dirToPurge, ttl)`
	`74`	`+ New(dirToPurge).Purge(ttl)`
`75`	`75`
`76`	`76`	`files, err := dirToPurge.Join("fresh").Stat()`
`77`	`77`	`require.Nil(t, err)`

`‎commands/compile/compile.go‎`

Lines changed: 3 additions & 3 deletions

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)`
	`141`	`+ buildcache.New(builderCtx.BuildPath.Parent()).GetOrCreate(builderCtx.BuildPath.Base())`
`142`	`142`	`// cache is purged after compilation to not remove entries that might be required`
`143`	`143`	`defer maybePurgeBuildCache()`
`144`	`144`
`@@ -312,6 +312,6 @@ func maybePurgeBuildCache() {`
`312`	`312`	`}`
`313`	`313`	`inventory.Store.Set("build_cache.compilation_count_since_last_purge", 0)`
`314`	`314`	`cacheTTL := configuration.Settings.GetDuration("build_cache.ttl").Abs()`
`315`		`- buildcache.Purge(paths.TempDir().Join("arduino", "cores"), cacheTTL)`
`316`		`- buildcache.Purge(paths.TempDir().Join("arduino", "sketches"), cacheTTL)`
	`315`	`+ buildcache.New(paths.TempDir().Join("arduino", "cores")).Purge(cacheTTL)`
	`316`	`+ buildcache.New(paths.TempDir().Join("arduino", "sketches")).Purge(cacheTTL)`
`317`	`317`	`}`

0 commit comments

Comments

(0)

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Uh oh!

Commit d7efac1

File tree

3 files changed

3 files changed

`‎buildcache/build_cache.go‎`

`‎buildcache/build_cache_test.go‎`

`‎commands/compile/compile.go‎`

0 commit comments