trustwallet / wallet-core

Cross-platform, cross-blockchain wallet library.
https://developer.trustwallet.com/wallet-core
Apache License 2.0
2.79k stars 1.57k forks source link

fix: Dockerfile build #4013

Closed robtry closed 3 weeks ago

robtry commented 3 weeks ago

Description

Fixed the Dockerfile, which previously failed to build and run tests correctly inside the Docker container. The changes ensure that the Docker environment is properly set up to handle both tasks without errors.

How to test

Build the Docker image using the command docker build -t wallet-core-dev . and verify that it completes without errors. docker run -i -t wallet-core-dev /bin/bash ./tools/build-and-test

Types of changes

Checklist

satoshiotomakan commented 3 weeks ago

Hi @robtry, the CI has failed. Also we do not have any issue with running docker build command

robtry commented 3 weeks ago

So can you run ? docker build docker/wallet-core --tag wallet-core-dev docker run -i -t wallet-core-dev /bin/bash

and

cd wallet-core
./tools/build-and-test

I mean the folder docker does not even exists