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

Fix build error due to unresolved merge conflict #2493

Merged
merged 1 commit into from
May 17, 2021

Conversation

fairclothjm
Copy link
Contributor

@fairclothjm fairclothjm commented May 17, 2021

My local build of delve which apparently pulls in the master branch started breaking. This is the culprit.

Error:

#9 75.27 # github.com/go-delve/delve/service/dap
#9 75.27 /usr/src/github.com/go-delve/delve/service/dap/server.go:107:1: syntax error: unexpected <<, expecting field name or embedded type
#9 75.27 /usr/src/github.com/go-delve/delve/service/dap/server.go:110:1: syntax error: unexpected ==, expecting field name or embedded type
#9 75.27 /usr/src/github.com/go-delve/delve/service/dap/server.go:113:1: syntax error: unexpected >>, expecting field name or embedded type

My local build of delve which apparently pulls in the master branch starting breaking. This is the culprit.

Error:
```
go-delve#9 75.27 # github.com/go-delve/delve/service/dap
go-delve#9 75.27 /usr/src/github.com/go-delve/delve/service/dap/server.go:107:1: syntax error: unexpected <<, expecting field name or embedded type
go-delve#9 75.27 /usr/src/github.com/go-delve/delve/service/dap/server.go:110:1: syntax error: unexpected ==, expecting field name or embedded type
go-delve#9 75.27 /usr/src/github.com/go-delve/delve/service/dap/server.go:113:1: syntax error: unexpected >>, expecting field name or embedded type
```
Copy link
Member

@derekparker derekparker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@derekparker
Copy link
Member

@fairclothjm that was quick! Hah, thanks for this. I must have accidentally merged something in with a conflict. The GH UI typically prevents this and I don't remember anything I pulled in failing CI, but... such is life.

Thanks for the patch.

@mylegfeelsfunny
Copy link

Thanks for getting this so quick

@derekparker
Copy link
Member

@polinasok @suzmue does this conflict resolution look correct to both of you?

This came as a result of merging #2444 which contained this conflict in a commit. That's my fault for merging.

@suzmue
Copy link
Contributor

suzmue commented May 17, 2021

LGTM.

Sorry about that.

@derekparker derekparker merged commit 5473887 into go-delve:master May 17, 2021
@fairclothjm fairclothjm deleted the patch-1 branch May 17, 2021 18:09
@fairclothjm
Copy link
Contributor Author

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants