don't fail on build error -- sometimes it can be irrelevant (e.g., pkg with empty top-level, but valid subdirs)

Randall C. O'Reilly · Randall C. O'Reilly · commit 18091317ea49 · 2020-11-23T17:31:26.000-08:00
diff --git a/gen.go b/gen.go
@@ -100,11 +100,10 @@ func loadPackage(path string, buildFirst bool) (*packages.Package, error) {
 
 		err = cmd.Run()
 		if err != nil {
-			log.Printf("error building [%s]: %v\n",
+			log.Printf("Note: there was an error building [%s] -- will continue but it may fail later: %v\n",
 				path,
 				err,
 			)
-			return nil, err
 		}
 	}
 
diff --git a/go.mod b/go.mod
@@ -3,7 +3,6 @@ module github.com/go-python/gopy
 go 1.13
 
 require (
-	github.com/goki/gide v1.0.10 // indirect
 	github.com/gonuts/commander v0.1.0
 	github.com/gonuts/flag v0.1.0
 	github.com/pkg/errors v0.9.1
diff --git a/go.sum b/go.sum

Original file line number	Diff line number	Diff line change
`@@ -100,11 +100,10 @@ func loadPackage(path string, buildFirst bool) (*packages.Package, error) {`
`100`	`100`
`101`	`101`	`err = cmd.Run()`
`102`	`102`	`if err != nil {`
`103`		`- log.Printf("error building [%s]: %v\n",`
	`103`	`+ log.Printf("Note: there was an error building [%s] -- will continue but it may fail later: %v\n",`
`104`	`104`	`path,`
`105`	`105`	`err,`
`106`	`106`	`)`
`107`		`- return nil, err`
`108`	`107`	`}`
`109`	`108`	`}`
`110`	`109`