Merge pull request #507 from LedgerHQ/fix/apa/standard_plugin_mk_with_ledger_assert
Fix standard plugin build with ledger_assert
This commit is contained in:
@@ -5,7 +5,25 @@
|
||||
| :rotating_light: | Breaks build |
|
||||
| :warning: | Breaks compatibility with app |
|
||||
|
||||
## [latest](/) - 2023/10/19
|
||||
## [latest](/) - 2023/12/06
|
||||
|
||||
### Fixed
|
||||
|
||||
* standard\_plugin build ([this PR on the SDK](https://github.com/LedgerHQ/ledger-secure-sdk/pull/473) had broken it)
|
||||
|
||||
### Changed
|
||||
|
||||
* utils renamed to plugin\_utils to prevent filename conflicts in plugins
|
||||
|
||||
## [4d8e044](/../../commit/4d8e044) - 2023/11/09
|
||||
|
||||
### Added
|
||||
|
||||
* standard\_plugin Makefile so plugins can use it & have a really small Makefile
|
||||
with only the relevant information
|
||||
* Comments in the plugin interface header file
|
||||
|
||||
## [1fe4085](/../../commit/1fe4085) - 2023/10/19
|
||||
|
||||
### Changed
|
||||
|
||||
|
||||
@@ -15,7 +15,7 @@
|
||||
* limitations under the License.
|
||||
*****************************************************************************/
|
||||
|
||||
#include "utils.h"
|
||||
#include "plugin_utils.h"
|
||||
|
||||
bool find_selector(uint32_t selector, const uint32_t *array, size_t size, size_t *idx) {
|
||||
for (size_t i = 0; i < size; ++i) {
|
||||
@@ -57,6 +57,8 @@ DISABLE_STANDARD_SNPRINTF = 1
|
||||
DISABLE_STANDARD_USB = 1
|
||||
DISABLE_STANDARD_WEBUSB = 1
|
||||
DISABLE_STANDARD_BAGL_UX_FLOW = 1
|
||||
DISABLE_DEBUG_LEDGER_ASSERT = 1
|
||||
DISABLE_DEBUG_THROW = 1
|
||||
|
||||
# Required for PRINTFs to compile
|
||||
ifeq ($(DEBUG),0)
|
||||
|
||||
@@ -210,8 +210,8 @@ if __name__ == "__main__":
|
||||
|
||||
files_to_copy = [
|
||||
"main.c",
|
||||
"utils.c",
|
||||
"utils.h",
|
||||
"plugin_utils.c",
|
||||
"plugin_utils.h",
|
||||
]
|
||||
for file in files_to_copy:
|
||||
shutil.copyfile("src_plugin_sdk/" + file,
|
||||
|
||||
Reference in New Issue
Block a user