-
Notifications
You must be signed in to change notification settings - Fork 78
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
Compile error after braking change in pkg/errors #74
Comments
arcana261
pushed a commit
to arcana261/logrus_sentry
that referenced
this issue
Jan 5, 2019
… github.com/pkg/errors See [Issue](evalphobia#74)
Is pkg/errors#188 related to this? |
@mexisme Yes dear good sir.. |
@arcana261 Thanks 🍡 #75 has been Merged. |
davecheney
added a commit
to pkg/errors
that referenced
this issue
Jan 9, 2019
Updates aws/aws-xray-sdk-go#77 Updates evalphobia/logrus_sentry#74 Go 1.12 has updated the behaviour of runtime.FuncForPC so that it behaves as it did in Go 1.11 and earlier. This allows errors.Frame to return to a uintptr representing the PC +1 of the caller. This will fix the build breakages of projects that were tracking HEAD of this package. Signed-off-by: Dave Cheney <[email protected]>
OMG They've done it again :'( :'( :'( |
I've again submitted another PR #76 which reverts us to using just PC's. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hiiiiiii
Today a breaking change in
github.com/pkg/errors
package was merged which changes type forerrors.Frame
from a simple program counter toruntime.Frame
.github.com/evalphobia/logrus_sentry/sentry.go:314:16
cannot convert stFrames[i] (type "github.com/pkg/errors".Frame) to type uintptr
I will work on a MR right away
The text was updated successfully, but these errors were encountered: