New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
cmd/vet: missing 'possible formatting directive in Fprintln call' warning #24646
Comments
What command line did you use to invoke |
I used the cl below. It works ok in linux but it does not in Macos
go vet <package>
…On Mon, Apr 2, 2018 at 4:30 PM Bryan C. Mills ***@***.***> wrote:
What command line did you use to invoke vet?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#24646 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AAq_2krpPlrvH-EmKBZtkbSk6tiM3REwks5tkopMgaJpZM4TD_Fw>
.
|
Please provide a complete example. |
Timed out in state WaitingForInfo. Closing. (I am just a bot, though. Please speak up if this is a mistake or you have the requested information.) |
A complete example is easy enough to construct from the original issue, and it appears to reproduce still:
|
Bad @gopherbot. |
Thanks for the complete reproducer, @josharian. I can reproduce the bug on Linux, so I'm going to look into a fix. |
Change https://golang.org/cl/111357 mentions this issue: |
- Disable `vet` for pkg/ksync/server/restart.go:57 (golang/go#24646) - Re-enable `gosec`
Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (
go version
)?go version go1.10.1 darwin/amd64
Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (
go env
)?GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/camarena/Library/Caches/go-build"
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/camarena/go"
GORACE=""
GOROOT="/Users/camarena/homebrew/Cellar/go/1.10.1/libexec"
GOTMPDIR=""
GOTOOLDIR="/Users/camarena/homebrew/Cellar/go/1.10.1/libexec/pkg/tool/darwin_amd64"
GCCGO="gccgo"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/22/p1vpwjb10wq7gcjp63w9ccdr00d83_/T/go-build934642368=/tmp/go-build -gno-record-gcc-switches -fno-common"
What did you do?
I had this method:
func (ccp CommandParser) WriteHelp(w io.Writer) {
fmt.Fprintln(w, "%T.WriteHelp Not implemented Yet", ccp)
}
it should be
func (ccp CommandParser) WriteHelp(w io.Writer) {
fmt.Fprintf(w, "%T.WriteHelp Not implemented Yet\n", ccp)
}
What did you expect to see?
I expect to see the message: "possible formatting directive in Fprintln call"
as it is reported in linux.
What did you see instead?
no message and no error reported on macos
The text was updated successfully, but these errors were encountered: