search

rust-mobile / cargo-subcommand

Library for implementing cargo subcommands
6 stars 6 forks source link

Read target-dir from cargo config file #7

Closed guilhermewerner closed 3 years ago

guilhermewerner commented 3 years ago

Read config from .cargo/config.toml

Fix an error where the target directory was hard coded, causing problems when a different target directory was used.

https://github.com/rust-windowing/android-ndk-rs/issues/102

MarijnS95 commented 3 years ago

For completeness I think this should probe and unify all .cargo/config.toml files in accordance to https://doc.rust-lang.org/cargo/reference/config.html#hierarchical-structure

Or should that be picked up as a separate PR?

guilhermewerner commented 3 years ago

I think it should be in another PR. As this already solves part of the problem, and it can definitely be improved, but it is still useful as an initial measure.