cosmos / ledger-cosmos-obsolete

Ledger Nano S support for Tendermint/Cosmos
Apache License 2.0
42 stars 29 forks source link

Error Upload the app to the device #156

Closed RNS135 closed 5 years ago

RNS135 commented 5 years ago

Unable to run this command after following all steps: "make load_cosmos # Loads Cosmos app (for users)" Output of this command is: make: *** No rule to make target 'load_cosmos'. Stop.

Please help.

saadbutt commented 5 years ago

We are facing a same issue after cmake -DDISABLE_DOCKER_BUILDS=ON . && make. @jleni is there any other way to upload app to the device?

jleni commented 5 years ago

There are actually two independent compilations...

Use cmake to run unit tests and develop on your host computer. That way it is possible to improve the decoder and other components without wasting time with the device.

Use make to build inside docker, etc. @RNS135 I think you are using the "internal" make file inside src/ledger. You should not use that. Instead run the Makefile in the root directory. That one launches a docker container with everything preinstalled when you use make build_cosmos and then you can use make load_cosmos

I hope this is clear enough. Feel free to send me more questions.

jleni commented 5 years ago

@RNS135 @saadbutt if you found that this could solve your problems.

It would be great (if you have 5 minutes) if you could send a PR with some quick changes to the docs so other people in the future do not get confused about these differences.

RNS135 commented 5 years ago

Sure @jleni, thanks

jleni commented 5 years ago

Is it now working? Should I close this issue then?