Skip to content
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

has tracing feature broken? #520

Closed
Slach opened this Issue Jan 28, 2018 · 2 comments

Comments

Projects
None yet
2 participants
@Slach
Copy link

commented Jan 28, 2018

I see something related error in #368
and try run tracing examples with -tags trace
What workaround i should use?

Steps for reproduce

go get -u -v github.com/mattn/go-sqlite3
go build -tags trace -o /tmp/trace_sqlite /home/ubuntu/go/src/github.com/mattn/go-sqlite3/_example/trace/main.go

Output

# command-line-arguments
_example/trace/main.go:98:15: cannot use uint(eventMask) (type uint) as type sqlite3._Ctype_uint in field 
value

Environment

GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/ubuntu/go"
GORACE=""
GOROOT="/usr/lib/go-1.9"
GOTOOLDIR="/usr/lib/go-1.9/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build859732878=/tmp/go-build -gno-record-gcc-switches"
CXX="g++"
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"

mattn added a commit that referenced this issue Jan 29, 2018

@mattn mattn closed this in #521 Jan 29, 2018

@Slach

This comment has been minimized.

Copy link
Author

commented Jan 29, 2018

@mattn wow ;) great, thanks for your great efforts !

could you explain me something?
why sqlite3_vtable.go and sqlite3_trace.go
need build with separate golang build tags?

this features make overhead when it's enabled by default?

@mattn

This comment has been minimized.

Copy link
Owner

commented Jan 29, 2018

this features make overhead when it's enabled by default?

yes, it make (bits) large binary.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.