Skip to content

go 1.14.4 #55639

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

Closed
wants to merge 2 commits into from
Closed

go 1.14.4 #55639

wants to merge 2 commits into from

Conversation

bayandin
Copy link
Member

@bayandin bayandin commented Jun 1, 2020

Created with brew bump-formula-pr.

@bayandin
Copy link
Member Author

bayandin commented Jun 2, 2020

Some fails:

brew install --build-from-source baidupcs-go
brew install --build-from-source charm
brew install --build-from-source dashing
brew install --build-from-source docker-swarm
brew install --build-from-source dvm
brew install --build-from-source gor
brew install --build-from-source perkeep
brew install --build-from-source source-to-image
brew install --build-from-source sync_gateway
brew install --build-from-source wellington
brew test --verbose aws-es-proxy
brew test --verbose cql
brew test --verbose dashing
brew test --verbose devd
brew test --verbose fn
brew test --verbose jabba
brew test --verbose mailhog

@dawidd6
Copy link
Contributor

dawidd6 commented Jun 2, 2020

Looks like the repository of baidupcs-go vanished: https://github.com/iikira/BaiduPCS-Go.git

It doesn't seem to be a dependency of anything though. Maybe we should remove it @Homebrew/core ?

@chenrui333
Copy link
Member

Looks like the repository of baidupcs-go vanished: https://github.com/iikira/BaiduPCS-Go.git

It doesn't seem to be a dependency of anything though. Maybe we should remove it @Homebrew/core ?

yeah, we should remove it, it is not rebuildable now.

@dawidd6
Copy link
Contributor

dawidd6 commented Jun 4, 2020

@bayandin could you please rebase on top of latest master?

@bayandin
Copy link
Member Author

bayandin commented Jun 5, 2020

As the last build failed on Catalina (No space left on device) and we need to rerun it anyway, I've separated fixes for failed tests (which is unrelated to the current go upgrade) to a separate PR: #55844

@bayandin bayandin force-pushed the go-1.14.4 branch 2 times, most recently from bcea802 to 93cc9ce Compare June 6, 2020 07:57
@magiconair
Copy link
Contributor

@bayandin It's been building for more than 6h. Is this normal?

image

@Bo98
Copy link
Member

Bo98 commented Jun 10, 2020

It is normal for Go.

@magiconair
Copy link
Contributor

For building Go on homebrew?

@Bo98
Copy link
Member

Bo98 commented Jun 10, 2020

And testing all formulae which depend on Go to make sure it works with the new version.

~11 hours is normal for Go.

@magiconair
Copy link
Contributor

I see. Thank you!

@roopakv
Copy link
Contributor

roopakv commented Jun 11, 2020

@Bo98 is there anything we can do to help push this along?

@alebcay
Copy link
Member

alebcay commented Jun 11, 2020

Current failed steps:

brew test --verbose annie
brew install --build-from-source charm
brew test --verbose darksky-weather 
brew install --build-from-source dashing
brew test --verbose dashing
brew install --build-from-source dvm
brew test --verbose faas-cli
brew install --build-from-source gor
brew test --verbose jabba
brew test --verbose jabba
brew install --build-from-source perkeep
brew install --build-from-source source-to-image
brew install --build-from-source sync_gateway
brew install --build-from-source wellington

Ideally, these would all be resolved before merging. Some of the failures are coupled (e.g. dashing - test will fail since build failed).

@roopakv
Copy link
Contributor

roopakv commented Jun 11, 2020

@alebcay #52184

if you notice there I have opened PRs for perkeep and wellington. If the repos are sort of in maintenance mode could we not block merging at least for those?

@Bo98
Copy link
Member

Bo98 commented Jun 11, 2020 via email

@roopakv
Copy link
Contributor

roopakv commented Jun 12, 2020

@Bo98 if you have a list we cannot skip, I am happy to start working on trying to fix them

@ryboe
Copy link

ryboe commented Jun 16, 2020

They finally cut a new release of charmstore-client, so if we update charm it should build with Go 1.14.

@roopakv
Copy link
Contributor

roopakv commented Jun 16, 2020

@Bo98 I feel like with every go release we come to packages that are broken and don't keep up.

What would a longer-term approach be here? use an internal version for homebrew downloads and builds and keep the external go keep moving along?

currently, charmstore has cut a release but quite a few other packages still have issues.

@MikeMcQuaid
Copy link
Member

@Bo98 I feel like with every go release we come to packages that are broken and don't keep up.

What would a longer-term approach be here? use an internal version for homebrew downloads and builds and keep the external go keep moving along?

This is "rolling release package management" in a nutshell. It's not really avoidable.

@SMillerDev
Copy link
Member

The best way forward that wouldn't just help Homebrew is to help other projects improve their CI and code. That way we can avoid running into these issues altogether.

@roopakv
Copy link
Contributor

roopakv commented Jun 16, 2020

hmmm thanks for the info @MikeMcQuaid & @SMillerDev :)

I don't really understand too much of the internals. Very naively I was sort of hoping we could do something like use [email protected] to build older breaking packages while moving go latest to be 1.14.4

But I also understand how that might become a nightmare with a lot of packages being left behind.

@atakanyenel
Copy link

I'm also following this PR for some time and I think one of the improvements could be to cache correctly building packages. That way , if a package fails , we don't have to wait for 10 hours in the next run. I'm also aware that implementing this with Github actions is not trivial.

@Bo98
Copy link
Member

Bo98 commented Jun 19, 2020

I'm sorry for not being around this week and following up here - I should have dealt with this pull request last week.

You are right that pre-existing failures are not a blocker for merging this.

I've gone through every single formula failing here and confirmed they are either already known as broken or not as a result of 1.14.4.

I will spend some time this weekend documenting these known broken formulae (or fixing the ones that can be fixed) so that it is more clearly known in the future.

@BrewTestBot
Copy link
Member

:shipit: @Bo98 has triggered a merge.

@MikeMcQuaid
Copy link
Member

I will spend some time this weekend documenting these known broken formulae (or fixing the ones that can be fixed) so that it is more clearly known in the future.

Thanks @Bo98! Dumped the basics into #56539

@bayandin bayandin deleted the go-1.14.4 branch June 19, 2020 23:24
@gromgit gromgit mentioned this pull request Aug 9, 2020
5 tasks
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.