Simplify cache directory creation

This commit is contained in:
greatroar 2020-10-05 10:22:05 +02:00
parent 3c6671b18b
commit 7d0fa1a686
2 changed files with 2 additions and 34 deletions

View File

@ -96,7 +96,7 @@ func New(id string, basedir string) (c *Cache, err error) {
}
}
created, err := mkdirCacheDir(basedir)
err = fs.MkdirAll(basedir, 0700)
if err != nil {
return nil, err
}
@ -119,6 +119,7 @@ func New(id string, basedir string) (c *Cache, err error) {
}
// create the repo cache dir if it does not exist yet
var created bool
_, err = fs.Lstat(cachedir)
if os.IsNotExist(err) {
err = fs.MkdirAll(cachedir, dirMode)

33
internal/cache/dir.go vendored
View File

@ -4,10 +4,6 @@ import (
"fmt"
"os"
"path/filepath"
"github.com/pkg/errors"
"github.com/restic/restic/internal/debug"
"github.com/restic/restic/internal/fs"
)
// DefaultDir returns $RESTIC_CACHE_DIR, or the default cache directory
@ -25,32 +21,3 @@ func DefaultDir() (cachedir string, err error) {
return filepath.Join(cachedir, "restic"), nil
}
// mkdirCacheDir ensures that the cache directory exists. It it didn't, created
// is set to true.
func mkdirCacheDir(cachedir string) (created bool, err error) {
var newCacheDir bool
fi, err := fs.Stat(cachedir)
if os.IsNotExist(errors.Cause(err)) {
err = fs.MkdirAll(cachedir, 0700)
if err != nil {
return true, errors.Wrap(err, "MkdirAll")
}
fi, err = fs.Stat(cachedir)
debug.Log("create cache dir %v", cachedir)
newCacheDir = true
}
if err != nil {
return newCacheDir, errors.Wrap(err, "Stat")
}
if !fi.IsDir() {
return newCacheDir, errors.Errorf("cache dir %v is not a directory", cachedir)
}
return newCacheDir, nil
}