diff options
| author | Vladimir Shushlin <v.shushlin@gmail.com> | 2019-10-22 09:48:22 +0300 |
|---|---|---|
| committer | Vladimir Shushlin <v.shushlin@gmail.com> | 2019-10-22 13:24:40 +0300 |
| commit | 181aa2c4619a15cefc1bd382a70a5d014f91a80a (patch) | |
| tree | 62b81e3d7b38a299db6f9480536cede949a68289 /Makefile.util.mk | |
| parent | d7f7542f856487e2692feec0a48f7a23851fb940 (diff) | |
Fix make verify ci job
For some reason long awk command makes it crash with a
segmentation fault error.
I tested this by replacing $(allfiles) in
`"Please run ./bin/goimports -w -local $(IMPORT_PATH) -l $(allfiles)"`
with strings of various length, and it fails on longer strings.
I haven't yet figured out the source of the problem, so just
copypasted the approach from workhorse - extracted a separate script
which uses if in shell script and echo to print the suggestion.
Diffstat (limited to 'Makefile.util.mk')
| -rw-r--r-- | Makefile.util.mk | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/Makefile.util.mk b/Makefile.util.mk index 8a36d805..80ca63b1 100644 --- a/Makefile.util.mk +++ b/Makefile.util.mk @@ -3,7 +3,7 @@ verify: list fmt vet lint complexity fmt: bin/goimports .GOPATH/.ok - $Q @./bin/goimports -local $(IMPORT_PATH) -l $(allfiles) | awk '{ print } END { if (NR>0) { print "Please run ./bin/goimports -w -local $(IMPORT_PATH) -l $(allfiles)"; exit 1 } }' + $Q @_support/validate-formatting.sh $(allfiles) vet: .GOPATH/.ok $Q go vet $(allpackages) |