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

go 1.14 #10467

Merged
merged 26 commits into from
Mar 20, 2020
Merged

go 1.14 #10467

merged 26 commits into from
Mar 20, 2020

Conversation

techknowlogick
Copy link
Member

@techknowlogick techknowlogick commented Feb 26, 2020

Note: This also increases the minimum supported version of go to 1.12

@techknowlogick techknowlogick added this to the 1.12.0 milestone Feb 26, 2020
@techknowlogick techknowlogick added the type/refactoring Existing code has been cleaned up. There should be no new functionality. label Feb 26, 2020
@techknowlogick techknowlogick added the status/blocked This PR cannot be merged yet, i.e. because it depends on another unmerged PR label Feb 26, 2020
sapk
sapk previously requested changes Feb 26, 2020
Copy link
Member

@sapk sapk left a comment

Choose a reason for hiding this comment

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

Please update the go.mod version to use latest go std lib.

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Feb 26, 2020
@silverwind
Copy link
Member

silverwind commented Feb 26, 2020

This also increases the minimum supported version of go to 1.12

Need to update Makefile, README.md, .drone.yml and docs with 1.11 -> 1.12.

@techknowlogick
Copy link
Member Author

techknowlogick commented Mar 3, 2020

Failing due to error described in #10552 (and a few other places)

@techknowlogick techknowlogick removed the status/blocked This PR cannot be merged yet, i.e. because it depends on another unmerged PR label Mar 3, 2020
@GiteaBot GiteaBot added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Mar 3, 2020
@techknowlogick
Copy link
Member Author

Current template that need resolution:

template: repo/issue/list:271:46: executing "repo/issue/list" at <(len .PullRequest.ConflictedFiles) gt 0>: can't give argument to non-function len .PullRequest.ConflictedFiles

@techknowlogick
Copy link
Member Author

Ok, now that template issues have been resolved, it seems that on ARM there are failures when TestAPIUpdateFile is run. I suspect it is related to indexing of code.

@6543
Copy link
Member

6543 commented Mar 5, 2020

#10597 got merged can you update/use goversion var ;

@jolheiser
Copy link
Member

Resolved conflicts. 👍

@lafriks
Copy link
Member

lafriks commented Mar 5, 2020

looks like some kind of deadlock in queues on arm

Makefile Outdated Show resolved Hide resolved
Copy link
Member

@sapk sapk left a comment

Choose a reason for hiding this comment

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

Removing my blocking review since this part has been fixed.

@sapk sapk dismissed their stale review March 10, 2020 11:04

Removing my blocking review since this part has been fixed.

@guillep2k
Copy link
Member

https://golang.org/doc/go1.14

A consequence of the implementation of preemption is that on Unix systems, including Linux and macOS systems, programs built with Go 1.14 will receive more signals than programs built with earlier releases. This means that programs that use packages like syscall or golang.org/x/sys/unix will see more slow system calls fail with EINTR errors. Those programs will have to handle those errors in some way, most likely looping to try the system call again. For more information about this see man 7 signal for Linux systems or similar documentation for other systems.

Should we be concerned about this? How many of our vendored packages use system calls?

@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Mar 11, 2020
@silverwind
Copy link
Member

@guillep2k

$ grep '"syscall"' vendor -r | wc -l
161
$ grep '"golang.org/x/sys/unix"' vendor -r | wc -l
11

@techknowlogick
Copy link
Member Author

ping LG-TM

@techknowlogick techknowlogick merged commit 2fa1078 into go-gitea:master Mar 20, 2020
@techknowlogick techknowlogick deleted the go-1.14 branch March 20, 2020 14:29
@go-gitea go-gitea locked and limited conversation to collaborators Nov 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. type/refactoring Existing code has been cleaned up. There should be no new functionality.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants