Skip to content

Update lucas-clemente/quic-go sum #6068

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

Update lucas-clemente/quic-go sum #6068

wants to merge 1 commit into from

Conversation

popey
Copy link
Contributor

@popey popey commented Oct 8, 2019

I was building an unrelated PR of syncthing, and the sum for lucas-clemente/quic-go needed updating. So doing this separately, another PR to follow.

Purpose

Fixes the build, which fails when the checksum for this component is wrong, which it is.

get "gopkg.in/ldap.v2": found meta tag get.metaImport{Prefix:"gopkg.in/ldap.v2", VCS:"git", RepoRoot:"https://gopkg.in/ldap.v2"} at https://gopkg.in/ldap.v2?go-get=1                                        
go: extracting gopkg.in/ldap.v2 v2.5.1                                                                                                                                                                       
go: downloading gopkg.in/asn1-ber.v1 v1.0.0-20181015200546-f715ec2f112d                                                                                                                                      
Fetching https://gopkg.in/asn1-ber.v1?go-get=1                                                                                                                                                               
verifying github.com/lucas-clemente/quic-go@v0.12.0: checksum mismatch                                                                                                                                       
        downloaded: h1:dYHUyB50gEQlK3KqytmNySzuyzAcaQ3iuI2ZReAfVrE=                                                                                                                                          
        go.sum:     h1:TRbvZ6F++sofeGbh+Z2IIyIOhl8KyGnYuA06g2yrHdI=                                                                                                                                          
exit status 1                                                                                                                                                                                                
exit status 1

Testing

I did a test build in a clean lxc container running Ubuntu 16.04.
go run build.go
Without this patch, the build fails.

Update lucas-clemente/quic-go sum
7189fb4 
I was building an unrelated PR of syncthing, and the sum for lucas-clemente/quic-go needed updating. So doing this separately, another PR to follow.
@AudriusButkevicius
Copy link
Member

There is a forum thread on this. This is a bug in go 1.12, upgrade to 1.13, nuke your module cache go clean -mod-cache and build again.

@popey
Copy link
Contributor Author

popey commented Oct 8, 2019

Thanks so much! This is my first contribution. Apologies for the noise.

@AudriusButkevicius
Copy link
Member

No worries, thanks for the effort.

@st-review st-review added the frozen-due-to-age Issues closed and untouched for a long time, together with being locked for discussion label Oct 8, 2020
@syncthing syncthing locked and limited conversation to collaborators Oct 8, 2020
imsodin added a commit to imsodin/syncthing that referenced this pull request Nov 19, 2020
@imsodin
all: Add configurable defaults (fixes syncthing#4224, fixes syncthing…
a161746 
…#6068)
imsodin added a commit to imsodin/syncthing that referenced this pull request Nov 20, 2020
@imsodin
all: Add configurable defaults (fixes syncthing#4224, fixes syncthing…
d92f8a8 
…#6068)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
frozen-due-to-age Issues closed and untouched for a long time, together with being locked for discussion
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@popey @AudriusButkevicius @st-review