declare UNUSED macro as it's declared in common SDK - Githubissues

LedgerHQ / nanos-secure-sdk

Secure (ST31) SDK for Ledger Nano S

Apache License 2.0

96 stars 54 forks source link

declare UNUSED macro as it's declared in common SDK #63

Closed ftheirs closed 7 months ago

ftheirs commented 9 months ago

I'm facing a build issue in the CI for Polkadot and this seems to be the problem. Just missing the verification if UNUSED is not declared.

https://github.com/LedgerHQ/ledger-secure-sdk/blob/e8b56c451ff9336a71fc746a5d9a1ef40a7d4855/include/os_helpers.h#L18-L20

xchapron-ledger commented 7 months ago

Done in https://github.com/LedgerHQ/nanos-secure-sdk/commit/8310d4e07e71400954ed6f9a9b31ffeac8b8d092#diff-dcf614b7117f99b12f6cf6354a860570f26c4b6b56881a3f0ee80e9f18509d2a