Support new ELF metadata + Add APDU dump with new Ledgerctl option

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

Support new ELF metadata + Add APDU dump with new Ledgerctl option #31

Closed agrojean-ledger closed 10 months ago

agrojean-ledger commented 1 year ago

Is is retro compatible with app where the api_level is an int and not a string? I guess we should try it to avoid issue during deployments until all apps uses the updated SDK.

I'll make it retro compatible. Thanks for noticing this potential issue. @xchapron-ledger

agrojean-ledger commented 1 year ago

@xchapron-ledger same thing goes for the section name. We should support section names with and without the .. :face_with_head_bandage:

agrojean-ledger commented 1 year ago

@xchapron-ledger I made sure to be retro compatible :

Tested with :

both .ledger.api_level and ledger.api_level (with or without prefix .).
both u8 or string storage for api_level.

@yhql FYI