Skip to content

Commit bd35379

Browse files
committed
chore: clean internal package
1 parent 2209bb3 commit bd35379

File tree

3 files changed

+25
-26
lines changed

3 files changed

+25
-26
lines changed

go.mod

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -75,7 +75,6 @@ require (
7575
github.com/nishanths/exhaustive v0.9.5
7676
github.com/nishanths/predeclared v0.2.2
7777
github.com/nunnatsa/ginkgolinter v0.8.1
78-
github.com/pkg/errors v0.9.1
7978
github.com/polyfloyd/go-errorlint v1.1.0
8079
github.com/quasilyte/go-ruleguard/dsl v0.3.22
8180
github.com/ryancurrah/gomodguard v1.3.0
@@ -154,6 +153,7 @@ require (
154153
github.com/olekukonko/tablewriter v0.0.5 // indirect
155154
github.com/pelletier/go-toml v1.9.5 // indirect
156155
github.com/pelletier/go-toml/v2 v2.0.5 // indirect
156+
github.com/pkg/errors v0.9.1 // indirect
157157
github.com/pmezard/go-difflib v1.0.0 // indirect
158158
github.com/power-devops/perfstat v0.0.0-20210106213030-5aafc221ea8c // indirect
159159
github.com/prometheus/client_golang v1.12.1 // indirect

internal/cache/cache.go

Lines changed: 10 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@ import (
1212
"bytes"
1313
"crypto/sha256"
1414
"encoding/hex"
15+
"errors"
1516
"fmt"
1617
"io"
1718
"os"
@@ -20,8 +21,6 @@ import (
2021
"strings"
2122
"time"
2223

23-
"github.com/pkg/errors"
24-
2524
"github.com/golangci/golangci-lint/internal/renameio"
2625
"github.com/golangci/golangci-lint/internal/robustio"
2726
)
@@ -80,7 +79,7 @@ func (c *Cache) fileName(id [HashSize]byte, key string) string {
8079
var errMissing = errors.New("cache entry not found")
8180

8281
func IsErrMissing(err error) bool {
83-
return errors.Cause(err) == errMissing
82+
return errors.Is(err, errMissing)
8483
}
8584

8685
const (
@@ -169,10 +168,10 @@ func (c *Cache) get(id ActionID) (Entry, error) {
169168
etime := entry[1 : 1+20]
170169
var buf [HashSize]byte
171170
if _, err = hex.Decode(buf[:], eid); err != nil || buf != id {
172-
return failed(errors.Wrapf(err, "failed to hex decode eid data in %s", fileName))
171+
return failed(fmt.Errorf("failed to hex decode eid data in %s: %w", fileName, err))
173172
}
174173
if _, err = hex.Decode(buf[:], eout); err != nil {
175-
return failed(errors.Wrapf(err, "failed to hex decode eout data in %s", fileName))
174+
return failed(fmt.Errorf("failed to hex decode eout data in %s: %w", fileName, err))
176175
}
177176
i := 0
178177
for i < len(esize) && esize[i] == ' ' {
@@ -192,7 +191,7 @@ func (c *Cache) get(id ActionID) (Entry, error) {
192191
}
193192

194193
if err = c.used(fileName); err != nil {
195-
return failed(errors.Wrapf(err, "failed to mark %s as used", fileName))
194+
return failed(fmt.Errorf("failed to mark %s as used: %w", fileName, err))
196195
}
197196

198197
return Entry{buf, size, time.Unix(0, tm)}, nil
@@ -264,15 +263,15 @@ func (c *Cache) used(file string) error {
264263
if os.IsNotExist(err) {
265264
return errMissing
266265
}
267-
return errors.Wrapf(err, "failed to stat file %s", file)
266+
return fmt.Errorf("failed to stat file %s: %w", file, err)
268267
}
269268

270269
if c.now().Sub(info.ModTime()) < mtimeInterval {
271270
return nil
272271
}
273272

274273
if err := os.Chtimes(file, c.now(), c.now()); err != nil {
275-
return errors.Wrapf(err, "failed to change time of file %s", file)
274+
return fmt.Errorf("failed to change time of file %s: %w", file, err)
276275
}
277276

278277
return nil
@@ -385,7 +384,7 @@ func (c *Cache) putIndexEntry(id ActionID, out OutputID, size int64, allowVerify
385384
return err
386385
}
387386
if err = os.Chtimes(file, c.now(), c.now()); err != nil { // mainly for tests
388-
return errors.Wrapf(err, "failed to change time of file %s", file)
387+
return fmt.Errorf("failed to change time of file %s: %w", file, err)
389388
}
390389

391390
return nil
@@ -443,7 +442,7 @@ func (c *Cache) copyFile(file io.ReadSeeker, out OutputID, size int64) error {
443442
if f, openErr := os.Open(name); openErr == nil {
444443
h := sha256.New()
445444
if _, copyErr := io.Copy(h, f); copyErr != nil {
446-
return errors.Wrap(copyErr, "failed to copy to sha256")
445+
return fmt.Errorf("failed to copy to sha256: %w", copyErr)
447446
}
448447

449448
f.Close()
@@ -519,7 +518,7 @@ func (c *Cache) copyFile(file io.ReadSeeker, out OutputID, size int64) error {
519518
return err
520519
}
521520
if err = os.Chtimes(name, c.now(), c.now()); err != nil { // mainly for tests
522-
return errors.Wrapf(err, "failed to change time of file %s", name)
521+
return fmt.Errorf("failed to change time of file %s: %w", name, err)
523522
}
524523

525524
return nil

internal/pkgcache/pkgcache.go

Lines changed: 14 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -4,12 +4,12 @@ import (
44
"bytes"
55
"encoding/gob"
66
"encoding/hex"
7+
"errors"
78
"fmt"
89
"runtime"
910
"sort"
1011
"sync"
1112

12-
"github.com/pkg/errors"
1313
"golang.org/x/tools/go/packages"
1414

1515
"github.com/golangci/golangci-lint/internal/cache"
@@ -61,7 +61,7 @@ func (c *Cache) Put(pkg *packages.Package, mode HashMode, key string, data inter
6161
err = gob.NewEncoder(buf).Encode(data)
6262
})
6363
if err != nil {
64-
return errors.Wrap(err, "failed to gob encode")
64+
return fmt.Errorf("failed to gob encode: %w", err)
6565
}
6666

6767
var aID cache.ActionID
@@ -71,21 +71,21 @@ func (c *Cache) Put(pkg *packages.Package, mode HashMode, key string, data inter
7171
if err == nil {
7272
subkey, subkeyErr := cache.Subkey(aID, key)
7373
if subkeyErr != nil {
74-
err = errors.Wrap(subkeyErr, "failed to build subkey")
74+
err = fmt.Errorf("failed to build subkey: %w", subkeyErr)
7575
}
7676
aID = subkey
7777
}
7878
})
7979
if err != nil {
80-
return errors.Wrapf(err, "failed to calculate package %s action id", pkg.Name)
80+
return fmt.Errorf("failed to calculate package %s action id: %w", pkg.Name, err)
8181
}
8282
c.ioSem <- struct{}{}
8383
c.sw.TrackStage("cache io", func() {
8484
err = c.lowLevelCache.PutBytes(aID, buf.Bytes())
8585
})
8686
<-c.ioSem
8787
if err != nil {
88-
return errors.Wrapf(err, "failed to save data to low-level cache by key %s for package %s", key, pkg.Name)
88+
return fmt.Errorf("failed to save data to low-level cache by key %s for package %s: %w", key, pkg.Name, err)
8989
}
9090

9191
return nil
@@ -101,13 +101,13 @@ func (c *Cache) Get(pkg *packages.Package, mode HashMode, key string, data inter
101101
if err == nil {
102102
subkey, subkeyErr := cache.Subkey(aID, key)
103103
if subkeyErr != nil {
104-
err = errors.Wrap(subkeyErr, "failed to build subkey")
104+
err = fmt.Errorf("failed to build subkey: %w", subkeyErr)
105105
}
106106
aID = subkey
107107
}
108108
})
109109
if err != nil {
110-
return errors.Wrapf(err, "failed to calculate package %s action id", pkg.Name)
110+
return fmt.Errorf("failed to calculate package %s action id: %w", pkg.Name, err)
111111
}
112112

113113
var b []byte
@@ -120,14 +120,14 @@ func (c *Cache) Get(pkg *packages.Package, mode HashMode, key string, data inter
120120
if cache.IsErrMissing(err) {
121121
return ErrMissing
122122
}
123-
return errors.Wrapf(err, "failed to get data from low-level cache by key %s for package %s", key, pkg.Name)
123+
return fmt.Errorf("failed to get data from low-level cache by key %s for package %s: %w", key, pkg.Name, err)
124124
}
125125

126126
c.sw.TrackStage("gob", func() {
127127
err = gob.NewDecoder(bytes.NewReader(b)).Decode(data)
128128
})
129129
if err != nil {
130-
return errors.Wrap(err, "failed to gob decode")
130+
return fmt.Errorf("failed to gob decode: %w", err)
131131
}
132132

133133
return nil
@@ -136,12 +136,12 @@ func (c *Cache) Get(pkg *packages.Package, mode HashMode, key string, data inter
136136
func (c *Cache) pkgActionID(pkg *packages.Package, mode HashMode) (cache.ActionID, error) {
137137
hash, err := c.packageHash(pkg, mode)
138138
if err != nil {
139-
return cache.ActionID{}, errors.Wrap(err, "failed to get package hash")
139+
return cache.ActionID{}, fmt.Errorf("failed to get package hash: %w", err)
140140
}
141141

142142
key, err := cache.NewHash("action ID")
143143
if err != nil {
144-
return cache.ActionID{}, errors.Wrap(err, "failed to make a hash")
144+
return cache.ActionID{}, fmt.Errorf("failed to make a hash: %w", err)
145145
}
146146
fmt.Fprintf(key, "pkgpath %s\n", pkg.PkgPath)
147147
fmt.Fprintf(key, "pkghash %s\n", hash)
@@ -167,7 +167,7 @@ func (c *Cache) packageHash(pkg *packages.Package, mode HashMode) (string, error
167167

168168
key, err := cache.NewHash("package hash")
169169
if err != nil {
170-
return "", errors.Wrap(err, "failed to make a hash")
170+
return "", fmt.Errorf("failed to make a hash: %w", err)
171171
}
172172

173173
fmt.Fprintf(key, "pkgpath %s\n", pkg.PkgPath)
@@ -176,7 +176,7 @@ func (c *Cache) packageHash(pkg *packages.Package, mode HashMode) (string, error
176176
h, fErr := cache.FileHash(f)
177177
<-c.ioSem
178178
if fErr != nil {
179-
return "", errors.Wrapf(fErr, "failed to calculate file %s hash", f)
179+
return "", fmt.Errorf("failed to calculate file %s hash: %w", f, fErr)
180180
}
181181
fmt.Fprintf(key, "file %s %x\n", f, h)
182182
}
@@ -199,7 +199,7 @@ func (c *Cache) packageHash(pkg *packages.Package, mode HashMode) (string, error
199199

200200
depHash, depErr := c.packageHash(dep, depMode)
201201
if depErr != nil {
202-
return errors.Wrapf(depErr, "failed to calculate hash for dependency %s with mode %d", dep.Name, depMode)
202+
return fmt.Errorf("failed to calculate hash for dependency %s with mode %d: %w", dep.Name, depMode, depErr)
203203
}
204204

205205
fmt.Fprintf(key, "import %s %s\n", dep.PkgPath, depHash)

0 commit comments

Comments
 (0)