search

OP-TEE / build

Makefiles to use OP-TEE on various platforms
106 stars 209 forks source link

optee_test_ext.mk: use the optee_test Makefile to build TAs #745

Closed jforissier closed 2 months ago

jforissier commented 2 months ago

Rather than enumerating all the TAs and invoking each Makefile individually (optee_test/ta//Makefile), user the higher level Makefile (optee_test/ta/Makefile.gmake). This makes things simpler here and less fragile if something changes in optee_test. In addition, this should fix an "Argument list too long" error that can happen depending on the environment [1]. Finally, TAs can now be built in parallel (make -jN).

Link: https://github.com/OP-TEE/build/issues/744 [1]

jforissier commented 2 months ago

Depends on https://github.com/OP-TEE/optee_test/pull/740.

jforissier commented 2 months ago

@jenswi-linaro are you OK with the fixup patch? I have tested this in combination with optee_test 740 and with the GP test suite, and it does work. I don't like very much the output paths, I would prefer a single out/ directory under the main optee_test tree but I see no easy way and I don't want to risk breaking builds for us or users of the optee_test Makefiles.

jenswi-linaro commented 2 months ago

Yes, looks good.