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

Does not work with cargo package renaming #197

Open
icefoxen opened this issue Jan 7, 2019 · 2 comments
Open

Does not work with cargo package renaming #197

icefoxen opened this issue Jan 7, 2019 · 2 comments

Comments

@icefoxen
Copy link

icefoxen commented Jan 7, 2019

Such as this line: https://github.com/alexheretic/glyph-brush/blob/master/glyph-brush/Cargo.toml#L16

Reproduction:

mkdir foo
cd foo
cargo init
echo 'glyph_brush = "0.2"' >> Cargo.toml
cargo build

Output:

> docker run --rm -v "$(pwd):/root/src" -w /root/src tomaka/cargo-apk cargo apk build
Compiling android_native_app_glue.c
Compiling injected-glue
Compiling glue_obj
Compiling crate
warning: unused manifest key: package.edition
    Updating registry `https://github.com/rust-lang/crates.io-index`
    Updating registry `https://github.com/rust-lang/crates.io-index`
    Updating registry `https://github.com/rust-lang/crates.io-index`
error: no matching package named `full_rusttype` found (required by `glyph_brush`)
location searched: registry https://github.com/rust-lang/crates.io-index
version required: ^0.7
@est31
Copy link

est31 commented Jan 14, 2019

warning: unused manifest key: package.edition

This reads like too old cargo. I'd suggest using cargo-apk from git which has an updated version of cargo.

@philip-alldredge
Copy link
Contributor

As mentioned by @est31, this has been resolved in the git version.

The current git now references a new docker image which is more up to date and will resolve the issue:
docker run --rm -v "$(pwd):/root/src" -w /root/src philipalldredge/cargo-apk cargo apk build

See #221.

Recommend closing if this resolves the issue.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants