Merge pull request #8 from Zondax/light_app

Enable light app mode

Author: jleni

.circleci/config.yml

@@ -52,20 +52,20 @@ jobs:
       # Docker entrypoint is not considered
       - run: git submodule update --init --recursive
       - run:
-          name: Build Standard app
+          name: Build Standard light parser app
           command: |
             source /home/zondax/.cargo/env
             make
       - run:
-          name: Build SR25519 app
+          name: Build Standard full parser app
           command: |
             source /home/zondax/.cargo/env
-            SUPPORT_SR25519=1 make
+            SUBSTRATE_PARSER_FULL=1 make
       - run:
-          name: Build Ledgeracio
+          name: Build SR25519 app
           command: |
             source /home/zondax/.cargo/env
-            COIN=Ledgeracio make
+            SUPPORT_SR25519=1 make
 
   test_zemu:
     machine:
@@ -94,9 +94,9 @@ jobs:
       - run:
           name: Build Ledger app
           command: |
-            make buildS
+            SUBSTRATE_PARSER_FULL=1 make buildS
             make clean
-            make buildX
+            SUBSTRATE_PARSER_FULL=1 make buildX
       - run:
           name: Build/Install build js deps
           command: |

.github/workflows/main.yml

@@ -108,9 +108,9 @@ jobs:
           npm install -g yarn
       - name: Build Ledger app
         run: |
-          make buildS
+          make build_full_parser_s
           make clean
-          make buildX
+          make build_full_parser_x
       - name: Build/Install build js deps
         run: |
           export PATH=~/.cargo/bin:$PATH

CMakeLists.txt

@@ -29,6 +29,7 @@ string(APPEND CMAKE_CXX_FLAGS " -fno-omit-frame-pointer -g")
 string(APPEND CMAKE_LINKER_FLAGS " -fno-omit-frame-pointer -g")
 
 add_definitions(-DAPP_STANDARD)
+add_definitions(-DSUBSTRATE_PARSER_FULL)
 
 if(ENABLE_FUZZING)
     add_definitions(-DFUZZING_BUILD_MODE_UNSAFE_FOR_PRODUCTION=1)
@@ -113,7 +114,6 @@ file(GLOB_RECURSE LIB_SRC
         ${CMAKE_CURRENT_SOURCE_DIR}/app/src/parser_impl.c
         ${CMAKE_CURRENT_SOURCE_DIR}/app/src/parser_txdef.c
         ${CMAKE_CURRENT_SOURCE_DIR}/app/src/substrate*.c
-        ${CMAKE_CURRENT_SOURCE_DIR}/app/src/substrate*.h
         )
 
 add_library(app_lib STATIC ${LIB_SRC})

Makefile

@@ -38,6 +38,12 @@ build_sr25519: buildS
 	cp $(CURDIR)/app/bin/app.elf $(CURDIR)/app/output/app_sr25519.elf
 	cp $(CURDIR)/app/bin/app.elf $(CURDIR)/app/bin/app_sr25519.elf
 
+build_full_parser_s: SUBSTRATE_PARSER_FULL=1
+build_full_parser_s: buildS
+
+build_full_parser_x: SUBSTRATE_PARSER_FULL=1
+build_full_parser_x: buildX
+
 tests_tools_build:
 	cd tests_tools/neon && yarn install
 

app/Makefile

@@ -53,6 +53,11 @@ DEFINES += SUPPORT_SR25519
 $(info ************  SR25519 ENABLED ************)
 endif
 
+ifeq ($(SUBSTRATE_PARSER_FULL),1)
+DEFINES += SUBSTRATE_PARSER_FULL
+$(info ************ FULL PARSER ENABLED ************)
+endif
+
 ifndef COIN
 COIN=RAD
 endif

app/Makefile.version

@@ -3,4 +3,4 @@ APPVERSION_M=1
 # This is the `spec_version` field of `Runtime`
 APPVERSION_N=240
 # This is the `impl_version` field of `Runtime`
-APPVERSION_P=0
+APPVERSION_P=1