Skip to content

(ci): fix cross compilation #6516

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
Merged
merged 1 commit into from
Jul 26, 2023
Merged

(ci): fix cross compilation #6516

merged 1 commit into from
Jul 26, 2023

Conversation

everettraven
Copy link
Contributor

Description of the change:

  • Sets the environment variable CARGO_NET_GIT_FETCH_WITH_CLI=true for rust cross compilation

Motivation for the change:

Checklist

If the pull request includes user-facing changes, extra documentation is required:

@everettraven
(ci): fix cross compilation
e48315e 
by setting CARGO_NET_GIT_FETCH_WITH_CLI=true

Signed-off-by: Bryce Palmer <bpalmer@redhat.com>
@openshift-ci openshift-ci bot requested review from jberkhahn and oceanc80 July 26, 2023 13:17
@everettraven everettraven temporarily deployed to deploy July 26, 2023 13:18 — with GitHub Actions Inactive
@everettraven everettraven temporarily deployed to deploy July 26, 2023 13:18 — with GitHub Actions Inactive
@everettraven everettraven temporarily deployed to deploy July 26, 2023 13:18 — with GitHub Actions Inactive
@everettraven everettraven temporarily deployed to deploy July 26, 2023 13:18 — with GitHub Actions Inactive
@everettraven everettraven temporarily deployed to deploy July 26, 2023 13:18 — with GitHub Actions Inactive
@everettraven everettraven temporarily deployed to deploy July 26, 2023 13:18 — with GitHub Actions Inactive
rashmigottipati
rashmigottipati approved these changes Jul 26, 2023
View reviewed changes
Copy link
Member

@rashmigottipati rashmigottipati left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Jul 26, 2023
@oceanc80
Copy link
Collaborator

/lgtm

@everettraven everettraven merged commit 7d83efe into operator-framework:master Jul 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rashmigottipati rashmigottipati rashmigottipati approved these changes

@jberkhahn jberkhahn Awaiting requested review from jberkhahn

@oceanc80 oceanc80 Awaiting requested review from oceanc80

Assignees

@rashmigottipati rashmigottipati

@oceanc80 oceanc80

Labels

lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@everettraven @oceanc80 @rashmigottipati