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

[Merged by Bors] - Fixed cross-compiling by replacing wget with curl #1759

Closed
wants to merge 1 commit into from

Conversation

ethDreamer
Copy link
Member

It looks like the default docker image used by cross doesn't have
wget installed. This causes builds to fail. This can be fixed by
switching to curl.

Issue Addressed

cross-compiling was broken (at least for build-aarch64)

Proposed Changes

swap wget for curl

It looks like the default docker image used by cross doesn't have
wget installed. This causes builds to fail. This can be fixed by
switching to curl.
Copy link
Member

@michaelsproul michaelsproul left a comment

Choose a reason for hiding this comment

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

We must have gotten lucky with caching when we built the official images for release, I agree this is much safer. I also think avoiding the dependency on wget is generally desirable

bors r+

bors bot pushed a commit that referenced this pull request Oct 11, 2020
It looks like the default docker image used by cross doesn't have
wget installed. This causes builds to fail. This can be fixed by
switching to curl.

## Issue Addressed
cross-compiling was broken (at least for build-aarch64)

## Proposed Changes
swap wget for curl
@bors bors bot changed the title Fixed cross-compiling by replacing wget with curl [Merged by Bors] - Fixed cross-compiling by replacing wget with curl Oct 12, 2020
@bors bors bot closed this Oct 12, 2020
@ethDreamer ethDreamer deleted the cross_compile_wget_fix branch December 19, 2022 21:31
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.

2 participants