search

LedgerHQ / cargo-ledger

(Moved in the ledger device rust SDK --> https://github.com/LedgerHQ/ledger-device-rust-sdk). Load code on a Ledger device with a `cargo` subcommand
Apache License 2.0
15 stars 12 forks source link

Add env vars for objcopy and size #3

Closed Ericson2314 closed 3 years ago

Ericson2314 commented 3 years ago

We build our own toolchain, which have more specific names than arm-none-eabi-... because they are just for the ledger, we therefore need some way to un-hardcode these names in cargo-ledger.

As discussed in #2, they env vars are long, but designed to follow the conventions Cargo already established. See https://doc.rust-lang.org/cargo/reference/config.html#environment-variables for details.

The fallback if the env vars are not defined should ensure that no one's workflow is broken.

yhql commented 3 years ago

Thanks for the fix and the link ! I did not know you could use env vars also for this specific stuff.