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

disable building with go1.10 for release #4785

Closed
wants to merge 1 commit into from

Conversation

whyrusleeping
Copy link
Member

Something in go1.10 breaks ipfs in some way. This prevents people from building the release on their own and complaining when things break. We will remove this as soon as the 0.4.15 cycle starts.

License: MIT
Signed-off-by: Jeromy [email protected]

License: MIT
Signed-off-by: Jeromy <[email protected]>
@ghost ghost assigned whyrusleeping Mar 7, 2018
@ghost ghost added the status/in-progress In progress label Mar 7, 2018
@Kubuxu
Copy link
Member

Kubuxu commented Mar 7, 2018

Our whole CI uses Go1.10...

@victorb
Copy link
Member

victorb commented Mar 7, 2018

Our whole CI uses Go1.10...

We can have Jenkins build both go1.9 and go1.10 for you if you want? Easier to track progress

@Kubuxu
Copy link
Member

Kubuxu commented Mar 7, 2018

@victorbjelkholm unfortunately for some reason the problems are not showing up in our tests.

So for time being building with Go 1.10 won't help much.

@Kubuxu
Copy link
Member

Kubuxu commented Mar 7, 2018

looks like jenkins is running 1.9

@Kubuxu
Copy link
Member

Kubuxu commented Mar 7, 2018

@whyrusleeping looks like that build tag didin't work

@Kubuxu Kubuxu mentioned this pull request Mar 7, 2018
@whyrusleeping
Copy link
Member Author

@Kubuxu it looks like it did work. All the tests failed

@Kubuxu
Copy link
Member

Kubuxu commented Mar 8, 2018

Yes but Jenkins is using Go 1.9 on master. Otherwise we would have noticed long time ago as there was one more test failing apart from one I've added in #4790

@victorb
Copy link
Member

victorb commented Mar 8, 2018

unfortunately for some reason the problems are not showing up in our tests

As I understand, the issue applies to very basic usage of get, which should surely be caught in your tests? (at least as I understand #4781). If this issue only happens with 1.10 and not with 1.9, tests would be passing on 1.9 and with this fix, not be runnable at all on 1.10 and with the other fixes (#4793) should be passing.

@Kubuxu
Copy link
Member

Kubuxu commented Mar 8, 2018

@victorbjelkholm in short terms yes.

@Kubuxu
Copy link
Member

Kubuxu commented Mar 8, 2018

@victorbjelkholm main fix is #4790 the #4793 is fixing the patter in other places across the codebase but as far as we know nothing else is broken.

The bug arose because of bad usage of Read on our side and changes Golang made to tar standard library in Go 1.10.

@Kubuxu Kubuxu removed the status/in-progress In progress label Mar 9, 2018
@Kubuxu Kubuxu closed this Mar 12, 2018
@Stebalien Stebalien deleted the fix/disable-go10-build branch February 28, 2019 22:30
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.

3 participants