Skip to content
This repository has been archived by the owner on Jul 24, 2024. It is now read-only.

Part of #6905, support for go 1.13.1, fixed openshift_funcs to handle new version string #6906

Closed
wants to merge 1 commit into from

Conversation

dhirajsb
Copy link
Contributor

No description provided.

…d openshift_funcs to handle new version string from crc oc client
@dhirajsb dhirajsb requested a review from lgarciaaco October 15, 2019 04:19
@pure-bot pure-bot bot added the pr/review-requested Use this if you want to have a review. pure-bot will prevent merging if set and no review given label Oct 15, 2019
@dhirajsb dhirajsb changed the title Fixes #6905, fixed error when building with go 1.13.1, fixed openshift_funcs to handle new version string from crc oc client Part of #6905, fixed error when building with go 1.13.1, fixed openshift_funcs to handle new version string from crc oc client Oct 15, 2019
@dhirajsb dhirajsb requested a review from christophd October 15, 2019 16:32
@dhirajsb dhirajsb changed the title Part of #6905, fixed error when building with go 1.13.1, fixed openshift_funcs to handle new version string from crc oc client Part of #6905, support for go 1.13.1, fixed openshift_funcs to handle new version string Oct 15, 2019
@@ -51,4 +51,6 @@ replace (
sigs.k8s.io/controller-tools => sigs.k8s.io/controller-tools v0.1.11-0.20190411181648-9d55346c2bde
)

replace git.apache.org/thrift.git => github.com/apache/thrift v0.12.0
replace git.apache.org/thrift.git => github.com/apache/thrift v0.0.0-20180902110319-2566ecd5d999
Copy link
Member

Choose a reason for hiding this comment

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

Not sure about this, if we used v0.12.0 and now we're switching to apache/thrift@2566ecd5d999, we would miss 68 commits.

@heiko-braun
Copy link
Collaborator

I had the same issue with go 1.13 and switched back to 1.1.2. What's the benefit of using 1.13 over 1.12?

@dhirajsb
Copy link
Contributor Author

Similar changes merged from other PRs

@dhirajsb dhirajsb closed this Oct 21, 2019
@dhirajsb dhirajsb deleted the crc-updates branch October 21, 2019 17:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
pr/review-requested Use this if you want to have a review. pure-bot will prevent merging if set and no review given size/XS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants