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

Native RustTLS not supported on android build? #1969

Open
NicolasDP opened this issue Mar 26, 2020 · 2 comments
Open

Native RustTLS not supported on android build? #1969

NicolasDP opened this issue Mar 26, 2020 · 2 comments
Assignees
Labels
A-build-android Area: Build system (Android) A-ci Area: Continuous Integration Priority - Medium technical debt issue to track potential technical dept we are introducing but will need care later.
Milestone

Comments

@NicolasDP
Copy link
Contributor

Describe the bug

On the latest release we canno longer build the ARVM android build: https://github.com/input-output-hk/jormungandr/runs/536061370?check_suite_focus=true

The issue is:

error[E0432]: unresolved import `platform`
##[error]  --> /cargo/registry/src/github.com-1ecc6299db9ec823/rustls-native-certs-0.1.0/src/lib.rs:22:9
   |
22 | pub use platform::load_native_certs;
   |         ^^^^^^^^ use of undeclared type or module `platform`

Mandatory Information

  1. jcli --full-version: v0.8.16;
  2. jormungandr --full-version v0.8.16;

To Reproduce

Steps are described here: https://github.com/input-output-hk/jormungandr/runs/536061370?check_suite_focus=true

Expected behavior

To compile

Additional context

recent change introduced from #1961

@NicolasDP NicolasDP added technical debt issue to track potential technical dept we are introducing but will need care later. Priority - Medium A-ci Area: Continuous Integration A-build-android Area: Build system (Android) labels Mar 26, 2020
@eugene-babichenko
Copy link
Contributor

eugene-babichenko commented Mar 26, 2020

Oops, seems so. rustls/rustls-native-certs#3

I will look at what we can do.

@eugene-babichenko
Copy link
Contributor

The fix was merged in reqwest, waiting for a new release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-build-android Area: Build system (Android) A-ci Area: Continuous Integration Priority - Medium technical debt issue to track potential technical dept we are introducing but will need care later.
Projects
None yet
Development

No branches or pull requests

2 participants