From 5a2e02635aade3d83eab18d800c093f9cdc5345b Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Manuel=20P=C3=A9gouri=C3=A9-Gonnard?= Date: Mon, 23 Jan 2023 12:51:52 +0100 Subject: [PATCH] Improve a few comments & documentation MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Signed-off-by: Manuel Pégourié-Gonnard --- docs/architecture/psa-migration/outcome-analysis.sh | 4 ++-- docs/architecture/psa-migration/strategy.md | 2 +- tests/scripts/all.sh | 4 ++-- 3 files changed, 5 insertions(+), 5 deletions(-) diff --git a/docs/architecture/psa-migration/outcome-analysis.sh b/docs/architecture/psa-migration/outcome-analysis.sh index 6603c3ae5b..b26963b90f 100755 --- a/docs/architecture/psa-migration/outcome-analysis.sh +++ b/docs/architecture/psa-migration/outcome-analysis.sh @@ -10,8 +10,8 @@ # # USAGE: # - First, commit any uncommited changes. (Also, see warning below.) -# - including ssl-opt.sh: docs/architecture/psa-migration/outcome-analysis.sh -# - or: SKIP_SSL_OPT=1 docs/architecture/psa-migration/outcome-analysis.sh +# - Then launch --> [SKIP_SSL_OPT=1] docs/architecture/psa-migration/outcome-analysis.sh +# - SKIP_SSL_OPT=1 can optionally be set to skip ssl-opt.sh tests # # WARNING: this script checks out a commit other than the head of the current # branch; it checks out the current branch again when running successfully, diff --git a/docs/architecture/psa-migration/strategy.md b/docs/architecture/psa-migration/strategy.md index 07fc48859f..1542324746 100644 --- a/docs/architecture/psa-migration/strategy.md +++ b/docs/architecture/psa-migration/strategy.md @@ -393,7 +393,7 @@ manually for each PR that changes dependency declarations by using the script 2. The set of tests skipped in the driver-only build is the same as in an equivalent software-based configuration. This is tested automatically by the CI in the "Results analysis" stage, by running -`tests/scripts/analyze_outcomes.csv`. See the +`tests/scripts/analyze_outcomes.py`. See the `analyze_driver_vs_reference_xxx` actions in the script and the comments above their declaration for how to do that locally. diff --git a/tests/scripts/all.sh b/tests/scripts/all.sh index ff1efe7955..73191b64df 100755 --- a/tests/scripts/all.sh +++ b/tests/scripts/all.sh @@ -2053,8 +2053,8 @@ component_test_psa_crypto_config_accel_ecdsa_use_psa () { loc_accel_flags=$( echo "$loc_accel_list" | sed 's/[^ ]* */-DLIBTESTDRIVER1_MBEDTLS_PSA_ACCEL_&/g' ) make -C tests libtestdriver1.a CFLAGS="$ASAN_CFLAGS $loc_accel_flags" LDFLAGS="$ASAN_CFLAGS" - # Configure and build the test driver library - # ------------------------------------------- + # Configure and build the main libraries with drivers enabled + # ----------------------------------------------------------- # Use the same config as reference, only without built-in ECDSA config_psa_crypto_config_ecdsa_use_psa 1