Great to hear that that.

BTW, instead of building all targets using “install”

You can build a specific target separately by:

cmake --build cmake_build --target <target>

To list the targets available. use:

cmake --build cmake_build --target help

Cheers,

Anton

From: Christopher Brand <chris.brand@cypress.com>
Sent: 08 September 2020 22:30
To: Anton Komlev <Anton.Komlev@arm.com>; tf-m@lists.trustedfirmware.org
Cc: nd <nd@arm.com>
Subject: RE: TF-M build system update heads-up

Thanks, Anton. That does work.

The command I ran was definitely correct – I’ve tried switching back-and-forth between “-B” and “--build”, and they consistently fail and succeed, respectively. I’ve also seen my email client change “--” to “—“.

I’m running cmake 3.18.2 on Ubuntu 18.04.

Chris

From: TF-M <tf-m-bounces@lists.trustedfirmware.org> On Behalf Of Anton Komlev via TF-M
Sent: Tuesday, September 8, 2020 1:13 PM
To: 'tf-m@lists.trustedfirmware.org' <tf-m@lists.trustedfirmware.org>
Cc: nd <nd@arm.com>
Subject: Re: [TF-M] TF-M build system update heads-up

CAUTION: This email originated from outside of the organization. Do not click links or open attachments unless you recognize the sender and know the content is safe.

Hi Chris,

Could you try this:

cmake --build cmake_build -- install

Please note, when I copied your command the minus sign “-” was incorrect – probably a dash or hyphen sign. Need double check the doc.

Hope it helps,

Anton

From: TF-M <tf-m-bounces@lists.trustedfirmware.org> On Behalf Of Gyorgy Szing via TF-M
Sent: 08 September 2020 19:48
To: Christopher Brand <chris.brand@cypress.com>; tf-m@lists.trustedfirmware.org
Cc: nd <nd@arm.com>
Subject: Re: [TF-M] TF-M build system update heads-up

Hi,

Sorry to hear that. When you manage to find out the culprit please drop an email to the list. I am really curious.

/George

From: Christopher Brand <chris.brand@cypress.com>
Sent: 08 September 2020 19:08
To: Gyorgy Szing <Gyorgy.Szing@arm.com>; tf-m@lists.trustedfirmware.org
Cc: nd <nd@arm.com>
Subject: RE: TF-M build system update heads-up

Thanks, Gyorgy. I did indeed have the double dash (pretty sure I just copied and pasted from the doc). I guess my email client decided to “fix it in the mail message.

Chris

From: Gyorgy Szing <Gyorgy.Szing@arm.com>
Sent: Saturday, September 5, 2020 5:42 AM
To: Christopher Brand <chris.brand@cypress.com>; tf-m@lists.trustedfirmware.org
Cc: nd <nd@arm.com>
Subject: RE: TF-M build system update heads-up

CAUTION: This email originated from outside of the organization. Do not click links or open attachments unless you recognize the sender and know the content is safe.

Hi Chris,

You need two dashes before install.

Options with a single dash are processed by cmake. Options after a double dash (--) are passed over to the build tool (in this case gnumake). (See the cmake documentation [1].)

So:

cmake -B cmake_build -- install

is the correct command.

I attached an image above to work around potential readability issues of some font sets. I hope it gets trough.

[1] https://cmake.org/cmake/help/v3.0/manual/cmake.1.html

/George

From: TF-M <tf-m-bounces@lists.trustedfirmware.org> On Behalf Of Christopher Brand via TF-M
Sent: 05 September 2020 00:26
To: tf-m@lists.trustedfirmware.org
Subject: Re: [TF-M] TF-M build system update heads-up

Thanks, Anton.

I tried again today, and got further – the first cmake command seems to succeed, but the second failed.

I used

git fetch "https://review.trustedfirmware.org/TF-M/trusted-firmware-m" refs/changes/72/5472/1 && git checkout FETCH_HEAD

to get the tree, followed by

cmake -S . -B cmake_build -G"Unix Makefiles" -DTFM_PLATFORM=cypress/psoc64 -DCMAKE_TOOLCHAIN_FILE=toolchain_GNUARM.cmake -DCMAKE_BUILD_TYPE=debug

and then

cmake -B cmake_build – install

This error seems like something trivial:

CMake Error: The source directory “…/trusted-firmware-m/install" does not exist.

Chris

From: TF-M <tf-m-bounces@lists.trustedfirmware.org> On Behalf Of Anton Komlev via TF-M
Sent: Thursday, September 3, 2020 11:35 AM
To: 'tf-m@lists.trustedfirmware.org' <tf-m@lists.trustedfirmware.org>
Cc: nd <nd@arm.com>
Subject: Re: [TF-M] TF-M build system update heads-up

CAUTION: This email originated from outside of the organization. Do not click links or open attachments unless you recognize the sender and know the content is safe.

Hi Chris,

Thanks for trying. Please specify the build type explicitly: -DCMAKE_BUILD_TYPE=debug

It shall be defaulted to “MinSizeRel", but something went wrong. Will be fixed asap.

Good luck,

Anton

From: TF-M <tf-m-bounces@lists.trustedfirmware.org> On Behalf Of Christopher Brand via TF-M
Sent: 03 September 2020 19:10
To: tf-m@lists.trustedfirmware.org
Subject: Re: [TF-M] TF-M build system update heads-up

I tried to build PSoC64 with the new build system a couple of days ago, and wasn’t successful.

Following docs/getting_started/tfm_build_instruction.rst (the “Example: building TF-M for AN521 platform using GCC:” section), I ran cmake -S . -B cmake_build -G"Unix Makefiles" -DTFM_PLATFORM=cypress/psoc64 -DCMAKE_TOOLCHAIN_FILE=toolchain_GNUARM.cmake

That failed :

CMake Error at secure_fw/partitions/crypto/CMakeLists.txt:107 (if):
if given arguments: "STREQUAL" "DEBUG" "OR" "STREQUAL" "Debug" "OR" "STREQUAL" "debug" Unknown arguments specified

Looking at that file it seemed that the default for CMAKE_BUILD_TYPE wasn’t being applied, so I tried this command:

cmake -S . -B cmake_build -G"Unix Makefiles" -DTFM_PLATFORM=cypress/psoc64 -DCMAKE_TOOLCHAIN_FILE=toolchain_GNUARM.cmake -DCMAKE_BUILD_TYPE=MinSizeRel

That got further, but failed with:

CMake Error at secure_fw/partitions/crypto/CMakeLists.txt:115 (target_include_directories):
  Cannot specify include directories for target "mbedcrypto_crypto_service"
  which is not built by this project.CMake Error at secure_fw/partitions/crypto/CMakeLists.txt:120 (target_compile_definitions):
  Cannot specify compile definitions for target "mbedcrypto_crypto_service"
  which is not built by this project.CMake Error at secure_fw/partitions/crypto/CMakeLists.txt:130 (target_sources):
  Cannot specify sources for target "mbedcrypto_crypto_service" which is not
  built by this project.CMake Error at secure_fw/partitions/crypto/CMakeLists.txt:135 (target_link_libraries):
  Cannot specify link libraries for target "mbedcrypto_crypto_service" which
  is not built by this project.

At that point, I gave up.

Chris

From: TF-M <tf-m-bounces@lists.trustedfirmware.org> On Behalf Of Anton Komlev via TF-M
Sent: Thursday, September 3, 2020 10:04 AM
To: 'tf-m@lists.trustedfirmware.org' <tf-m@lists.trustedfirmware.org>
Cc: nd <nd@arm.com>
Subject: [TF-M] TF-M build system update heads-up

CAUTION: This email originated from outside of the organization. Do not click links or open attachments unless you recognize the sender and know the content is safe.

Hello,

As I mentioned in the Open Tech Forum today – we are close to merge the new build system to the master branch. Plan to start doing that at the beginning of the next week.

It might cause code freeze for several days and minor problems in corner cases after that. The intention is to clean it out before the next release in November.

All changes to merge are in feature-ux-buildsystem branch at the moment.

Please signal if it conflicts with your plans.

Regards,

Anton

This message and any attachments may contain confidential information from Cypress or its subsidiaries. If it has been received in error, please advise the sender and immediately delete this message.