www.github.com/golangci/golangci-lint.git@v1.10.1/pkg/golinters/govet.go (about) 1 package golinters 2 3 import ( 4 "context" 5 "fmt" 6 "go/ast" 7 "go/token" 8 "os" 9 "os/exec" 10 "strings" 11 "time" 12 13 "github.com/golangci/golangci-lint/pkg/fsutils" 14 "github.com/golangci/golangci-lint/pkg/goutils" 15 "github.com/golangci/golangci-lint/pkg/lint/linter" 16 "github.com/golangci/golangci-lint/pkg/logutils" 17 "github.com/golangci/golangci-lint/pkg/result" 18 "github.com/golangci/golangci-lint/pkg/timeutils" 19 govetAPI "github.com/golangci/govet" 20 ) 21 22 type Govet struct{} 23 24 func (Govet) Name() string { 25 return "govet" 26 } 27 28 func (Govet) Desc() string { 29 return "Vet examines Go source code and reports suspicious constructs, " + 30 "such as Printf calls whose arguments do not align with the format string" 31 } 32 33 func (g Govet) Run(ctx context.Context, lintCtx *linter.Context) ([]result.Issue, error) { 34 var govetIssues []govetAPI.Issue 35 var err error 36 if lintCtx.Settings().Govet.UseInstalledPackages { 37 govetIssues, err = g.runOnInstalledPackages(ctx, lintCtx) 38 if err != nil { 39 return nil, fmt.Errorf("can't run govet on installed packages: %s", err) 40 } 41 } else { 42 govetIssues, err = g.runOnSourcePackages(ctx, lintCtx) 43 if err != nil { 44 return nil, fmt.Errorf("can't run govet on source packages: %s", err) 45 } 46 } 47 48 if len(govetIssues) == 0 { 49 return nil, nil 50 } 51 52 res := make([]result.Issue, 0, len(govetIssues)) 53 for _, i := range govetIssues { 54 res = append(res, result.Issue{ 55 Pos: i.Pos, 56 Text: markIdentifiers(i.Message), 57 FromLinter: g.Name(), 58 }) 59 } 60 return res, nil 61 } 62 63 func (g Govet) runOnInstalledPackages(ctx context.Context, lintCtx *linter.Context) ([]govetAPI.Issue, error) { 64 if err := g.installPackages(ctx, lintCtx); err != nil { 65 return nil, fmt.Errorf("can't install packages (it's required for govet): %s", err) 66 } 67 68 // TODO: check .S asm files: govet can do it if pass dirs 69 var govetIssues []govetAPI.Issue 70 for _, pkg := range lintCtx.PkgProgram.Packages() { 71 var astFiles []*ast.File 72 var fset *token.FileSet 73 for _, fname := range pkg.Files(lintCtx.Cfg.Run.AnalyzeTests) { 74 af := lintCtx.ASTCache.Get(fname) 75 if af == nil || af.Err != nil { 76 return nil, fmt.Errorf("can't get parsed file %q from ast cache: %#v", fname, af) 77 } 78 astFiles = append(astFiles, af.F) 79 fset = af.Fset 80 } 81 if len(astFiles) == 0 { 82 continue 83 } 84 issues, err := govetAPI.Analyze(astFiles, fset, nil, 85 lintCtx.Settings().Govet.CheckShadowing, getPath) 86 if err != nil { 87 return nil, err 88 } 89 govetIssues = append(govetIssues, issues...) 90 } 91 92 return govetIssues, nil 93 } 94 95 func (g Govet) installPackages(ctx context.Context, lintCtx *linter.Context) error { 96 inGoRoot, err := goutils.InGoRoot() 97 if err != nil { 98 return fmt.Errorf("can't check whether we are in $GOROOT: %s", err) 99 } 100 101 if inGoRoot { 102 // Go source packages already should be installed into $GOROOT/pkg with go distribution 103 lintCtx.Log.Infof("In $GOROOT, don't install packages") 104 return nil 105 } 106 107 if err := g.installNonTestPackages(ctx, lintCtx); err != nil { 108 return err 109 } 110 111 if err := g.installTestDependencies(ctx, lintCtx); err != nil { 112 return err 113 } 114 115 return nil 116 } 117 118 func (g Govet) installTestDependencies(ctx context.Context, lintCtx *linter.Context) error { 119 log := lintCtx.Log 120 packages := lintCtx.PkgProgram.Packages() 121 var testDirs []string 122 for _, pkg := range packages { 123 dir := pkg.Dir() 124 if dir == "" { 125 log.Warnf("Package %#v has empty dir", pkg) 126 continue 127 } 128 129 if !strings.HasPrefix(dir, ".") { 130 // go install can't work without that 131 dir = "./" + dir 132 } 133 134 if len(pkg.TestFiles()) != 0 { 135 testDirs = append(testDirs, dir) 136 } 137 } 138 139 if len(testDirs) == 0 { 140 log.Infof("No test files in packages %#v", packages) 141 return nil 142 } 143 144 args := append([]string{"test", "-i"}, testDirs...) 145 return runGoCommand(ctx, log, args...) 146 } 147 148 func (g Govet) installNonTestPackages(ctx context.Context, lintCtx *linter.Context) error { 149 log := lintCtx.Log 150 packages := lintCtx.PkgProgram.Packages() 151 var importPaths []string 152 for _, pkg := range packages { 153 if pkg.IsTestOnly() { 154 // test-only package will be processed by installTestDependencies 155 continue 156 } 157 158 dir := pkg.Dir() 159 if dir == "" { 160 log.Warnf("Package %#v has empty dir", pkg) 161 continue 162 } 163 164 if !strings.HasPrefix(dir, ".") { 165 // go install can't work without that 166 dir = "./" + dir 167 } 168 169 importPaths = append(importPaths, dir) 170 } 171 172 if len(importPaths) == 0 { 173 log.Infof("No packages to install, all packages: %#v", packages) 174 return nil 175 } 176 177 // we need type information of dependencies of analyzed packages 178 // so we pass -i option to install it 179 if err := runGoInstall(ctx, log, importPaths, true); err != nil { 180 // try without -i option: go < 1.10 doesn't support this option 181 // and install dependencies by default. 182 return runGoInstall(ctx, log, importPaths, false) 183 } 184 185 return nil 186 } 187 188 func runGoInstall(ctx context.Context, log logutils.Log, importPaths []string, withIOption bool) error { 189 args := []string{"install"} 190 if withIOption { 191 args = append(args, "-i") 192 } 193 args = append(args, importPaths...) 194 195 return runGoCommand(ctx, log, args...) 196 } 197 198 func runGoCommand(ctx context.Context, log logutils.Log, args ...string) error { 199 argsStr := strings.Join(args, " ") 200 defer timeutils.Track(time.Now(), log, "go %s", argsStr) 201 202 cmd := exec.CommandContext(ctx, "go", args...) 203 cmd.Env = append([]string{}, os.Environ()...) 204 cmd.Env = append(cmd.Env, "GOMAXPROCS=1") // don't consume more than 1 cpu 205 206 // use .Output but not .Run to capture StdErr in err 207 _, err := cmd.Output() 208 if err != nil { 209 var stderr string 210 if ee, ok := err.(*exec.ExitError); ok && ee.Stderr != nil { 211 stderr = ": " + string(ee.Stderr) 212 } 213 214 return fmt.Errorf("can't run [go %s]: %s%s", argsStr, err, stderr) 215 } 216 217 return nil 218 } 219 220 func filterFiles(files []*ast.File, fset *token.FileSet) []*ast.File { 221 newFiles := make([]*ast.File, 0, len(files)) 222 for _, f := range files { 223 if !goutils.IsCgoFilename(fset.Position(f.Pos()).Filename) { 224 newFiles = append(newFiles, f) 225 } 226 } 227 228 return newFiles 229 } 230 231 func (g Govet) runOnSourcePackages(_ context.Context, lintCtx *linter.Context) ([]govetAPI.Issue, error) { 232 // TODO: check .S asm files: govet can do it if pass dirs 233 var govetIssues []govetAPI.Issue 234 for _, pkg := range lintCtx.Program.InitialPackages() { 235 if len(pkg.Files) == 0 { 236 continue 237 } 238 239 filteredFiles := filterFiles(pkg.Files, lintCtx.Program.Fset) 240 issues, err := govetAPI.Analyze(filteredFiles, lintCtx.Program.Fset, pkg, 241 lintCtx.Settings().Govet.CheckShadowing, getPath) 242 if err != nil { 243 return nil, err 244 } 245 govetIssues = append(govetIssues, issues...) 246 } 247 248 return govetIssues, nil 249 } 250 251 func getPath(f *ast.File, fset *token.FileSet) (string, error) { 252 return fsutils.ShortestRelPath(fset.Position(f.Pos()).Filename, "") 253 }