C and CXX compiler identification unknown while building using docker

[kshitij01042002]

Describe the bug
When attempting to build wallet-core using the provided Docker image installation , the build process encounters an issue related to the C and C++ compilers.

To Reproduce

• Use the provided Docker image installation from the wallet-core website.
• Attempt to build Trust Wallet using the provided instructions.
• Observe the error mentioned above during the build process.

Expected behavior
The build shoud be successfull.

Screenshots

Additional context

Logs in ./wallet-core# cat build/CMakeFiles/CMakeError.log

[satoshiotomakan]

Hi @kshitij01042002, this error is unexpected if you do not modify Dockerfile.
Could you please provide the exact command you use to build the Docker image?

[kshitij01042002]

Hi @satoshiotomakan, thankyou for your response. I have not modified the Dockerfile.

I have used the steps mentioned here.

For the exact commands:

1. docker build docker/wallet-core --tag wallet-core-dev
2. docker run -i -t wallet-core-dev /bin/bash
3. ./tools/build-and-test

On running the command mentioned in the step 3 we get the following error.

[AnilChinchawale]

Hello @satoshiotomakan Can you share instructions that we can follow to verify test cases in the local environment.

Thanks.

[satoshiotomakan]

Found out the issue. The required environment variables are set in the Dockerfile as ENV command. They are valid during the Docker build only, so when you run the docker container with /bin/bash, the variables are no longer available.

wallet-core/Dockerfile

Lines 34 to 35 in b530432

	ENV CC=/usr/bin/clang-14
	ENV CXX=/usr/bin/clang++-14

wallet-core/Dockerfile

Line 40 in b530432

ENV PATH="/root/.cargo/bin:${PATH}"

So to run the tools/build-and-test command inside a docker container, you need to set the variables manually:

export CC=/usr/bin/clang-14
export CXX=/usr/bin/clang++-14
export PATH="/root/.cargo/bin:${PATH}"

./tools/build-and-test

However, the docker image you built with docker build has already compiled the static library and the test binary, so you can easily run the tests with

./build/tests/tests

[AnilChinchawale]

Thank you @satoshiotomakan , Let us try the above method quickly.

[kshitij01042002]

Hi @satoshiotomakan I tried by setting the variables inside the Docker container but the error still persist. Though I am able to run the tests inside the container using ./build/tests/tests and build using the command sudo docker build . --tag wallet-core-dev.