Leonardo Sandoval | 9dfdd1b | 2020-08-06 17:08:11 -0500 | [diff] [blame] | 1 | #!/usr/bin/env bash |
Fathi Boudra | 422bf77 | 2019-12-02 11:10:16 +0200 | [diff] [blame] | 2 | # |
Paul Sokolovsky | 422020c | 2024-06-18 23:21:07 +0300 | [diff] [blame] | 3 | # Copyright (c) 2019-2024 Arm Limited. All rights reserved. |
Fathi Boudra | 422bf77 | 2019-12-02 11:10:16 +0200 | [diff] [blame] | 4 | # |
| 5 | # SPDX-License-Identifier: BSD-3-Clause |
| 6 | # |
| 7 | |
| 8 | echo '----------------------------------------------' |
| 9 | echo '-- Running static checks on the source code --' |
| 10 | echo '----------------------------------------------' |
| 11 | |
| 12 | # Find the absolute path of the scripts' top directory |
| 13 | |
| 14 | cd "$(dirname "$0")/../.." |
| 15 | export CI_ROOT=$(pwd) |
| 16 | cd - |
| 17 | |
Paul Sokolovsky | 422020c | 2024-06-18 23:21:07 +0300 | [diff] [blame] | 18 | . $CI_ROOT/script/static-checks/common.sh |
| 19 | |
Harrison Mutai | 5d470c1 | 2025-07-16 08:26:58 +0000 | [diff] [blame^] | 20 | merge_base=$(get_merge_base) |
| 21 | if [[ -z "$merge_base" ]]; then |
| 22 | echo "Failed to find merge base, fetching entire change history" |
| 23 | |
| 24 | # Set GERRIT_REFSPEC if not already defined |
| 25 | if [[ -z "$GERRIT_REFSPEC" ]]; then |
| 26 | if [[ "$TF_GERRIT_PROJECT" == *tf-a-tests ]]; then |
| 27 | GERRIT_REFSPEC="$TFTF_GERRIT_REFSPEC" |
| 28 | else |
| 29 | GERRIT_REFSPEC="$TF_GERRIT_REFSPEC" |
| 30 | fi |
| 31 | fi |
| 32 | |
| 33 | git fetch --depth=100 origin "$GERRIT_REFSPEC" |
| 34 | git checkout FETCH_HEAD |
| 35 | |
| 36 | merge_base=$(get_merge_base) |
| 37 | |
| 38 | if [[ -z "$merge_base" ]]; then |
| 39 | echo "Failed to determine merge base after fetching. Exiting." >&2 |
| 40 | exit 1 |
| 41 | fi |
| 42 | fi |
| 43 | |
| 44 | export merge_base |
Fathi Boudra | 422bf77 | 2019-12-02 11:10:16 +0200 | [diff] [blame] | 45 | |
| 46 | export LOG_TEST_FILENAME=$(pwd)/static-checks.log |
| 47 | |
| 48 | echo |
| 49 | echo "###### Static checks ######" |
| 50 | echo |
| 51 | |
| 52 | echo "###### Static checks ######" > "$LOG_TEST_FILENAME" |
| 53 | echo >> "$LOG_TEST_FILENAME" |
| 54 | |
Paul Sokolovsky | 422020c | 2024-06-18 23:21:07 +0300 | [diff] [blame] | 55 | echo "Patch series being checked:" >> "$LOG_TEST_FILENAME" |
Harrison Mutai | 5d470c1 | 2025-07-16 08:26:58 +0000 | [diff] [blame^] | 56 | git log --oneline ${merge_base}..HEAD >> "$LOG_TEST_FILENAME" |
Paul Sokolovsky | 422020c | 2024-06-18 23:21:07 +0300 | [diff] [blame] | 57 | echo >> "$LOG_TEST_FILENAME" |
| 58 | echo "Base branch reference commit:" >> "$LOG_TEST_FILENAME" |
Harrison Mutai | 5d470c1 | 2025-07-16 08:26:58 +0000 | [diff] [blame^] | 59 | git log --oneline -1 ${merge_base} >> "$LOG_TEST_FILENAME" |
| 60 | |
Paul Sokolovsky | 422020c | 2024-06-18 23:21:07 +0300 | [diff] [blame] | 61 | |
| 62 | echo >> "$LOG_TEST_FILENAME" |
| 63 | |
Fathi Boudra | 422bf77 | 2019-12-02 11:10:16 +0200 | [diff] [blame] | 64 | # Reset error counters |
| 65 | |
| 66 | ERROR_COUNT=0 |
| 67 | WARNING_COUNT=0 |
| 68 | |
| 69 | # Ensure all the files contain a copyright |
| 70 | |
| 71 | echo 'Checking copyright in source files...' |
| 72 | echo |
| 73 | "$CI_ROOT"/script/static-checks/static-checks-check-copyright.sh . |
| 74 | if [ "$?" != 0 ]; then |
| 75 | echo "Copyright test: FAILURE" |
| 76 | ((ERROR_COUNT++)) |
| 77 | else |
| 78 | echo "Copyright test: PASS" |
| 79 | fi |
| 80 | echo |
| 81 | |
| 82 | # Check alphabetic order of headers included. |
| 83 | |
| 84 | if [ "$IS_CONTINUOUS_INTEGRATION" == 1 ]; then |
| 85 | "$CI_ROOT"/script/static-checks/static-checks-include-order.sh . patch |
| 86 | else |
| 87 | "$CI_ROOT"/script/static-checks/static-checks-include-order.sh . |
| 88 | fi |
| 89 | if [ "$?" != 0 ]; then |
| 90 | echo "Include order test: FAILURE" |
| 91 | ((WARNING_COUNT++)) |
| 92 | else |
| 93 | echo "Include order test: PASS" |
| 94 | fi |
| 95 | echo |
| 96 | |
| 97 | # Check line endings |
| 98 | |
Leonardo Sandoval | 239e8ac | 2020-07-27 15:14:59 -0500 | [diff] [blame] | 99 | if [ "$IS_CONTINUOUS_INTEGRATION" == 1 ]; then |
| 100 | "$CI_ROOT"/script/static-checks/static-checks-coding-style-line-endings.sh . patch |
| 101 | else |
| 102 | "$CI_ROOT"/script/static-checks/static-checks-coding-style-line-endings.sh |
| 103 | fi |
| 104 | |
Fathi Boudra | 422bf77 | 2019-12-02 11:10:16 +0200 | [diff] [blame] | 105 | if [ "$?" != 0 ]; then |
| 106 | echo "Line ending test: FAILURE" |
| 107 | ((ERROR_COUNT++)) |
| 108 | else |
| 109 | echo "Line ending test: PASS" |
| 110 | fi |
| 111 | echo |
| 112 | |
| 113 | # Check coding style |
| 114 | |
| 115 | echo 'Checking coding style compliance...' |
| 116 | echo |
| 117 | if [ "$IS_CONTINUOUS_INTEGRATION" == 1 ]; then |
| 118 | "$CI_ROOT"/script/static-checks/static-checks-coding-style.sh |
| 119 | else |
| 120 | "$CI_ROOT"/script/static-checks/static-checks-coding-style-entire-src-tree.sh |
| 121 | fi |
| 122 | if [ "$?" != 0 ]; then |
| 123 | echo "Coding style test: FAILURE" |
| 124 | ((ERROR_COUNT++)) |
| 125 | else |
| 126 | echo "Coding style test: PASS" |
| 127 | fi |
| 128 | echo |
| 129 | |
Zelalem | 219df41 | 2020-05-17 19:21:20 -0500 | [diff] [blame] | 130 | # Check for any Banned API usage |
| 131 | |
| 132 | echo 'Checking Banned API usage...' |
| 133 | echo |
| 134 | if [ "$IS_CONTINUOUS_INTEGRATION" == 1 ]; then |
| 135 | "$CI_ROOT"/script/static-checks/static-checks-banned-apis.sh . patch |
| 136 | else |
| 137 | "$CI_ROOT"/script/static-checks/static-checks-banned-apis.sh |
| 138 | fi |
| 139 | if [ "$?" != 0 ]; then |
| 140 | echo "Banned API check: FAILURE" |
| 141 | ((ERROR_COUNT++)) |
| 142 | else |
| 143 | echo "Banned API check: PASS" |
| 144 | fi |
| 145 | echo |
| 146 | |
Jayanth Dodderi Chidanand | 5132cb1 | 2021-08-09 17:54:47 +0100 | [diff] [blame] | 147 | # Check to ensure newly added source files are detected for Coverity Scan analysis |
| 148 | |
Jayanth Dodderi Chidanand | 253b339 | 2021-09-10 12:40:00 +0100 | [diff] [blame] | 149 | # Check to be executed only on trusted-firmware repository. |
Manish V Badarkhe | 8b846f4 | 2021-10-13 15:43:58 +0100 | [diff] [blame] | 150 | if [ "$REPO_UNDER_TEST" = "trusted-firmware" ] || [ "$REPO_UNDER_TEST" = "trusted-firmware-a" ]; then |
Jayanth Dodderi Chidanand | 253b339 | 2021-09-10 12:40:00 +0100 | [diff] [blame] | 151 | echo 'Checking whether the newly added source files are detected for Coverity Scan analysis...' |
| 152 | echo |
| 153 | "$CI_ROOT"/script/static-checks/static-checks-detect-newly-added-files.sh |
| 154 | if [ "$?" != 0 ]; then |
| 155 | echo "Files Detection check: FAILURE" |
| 156 | ((ERROR_COUNT++)) |
| 157 | else |
| 158 | echo "Files Detection check: PASS" |
| 159 | fi |
| 160 | echo |
Jayanth Dodderi Chidanand | 5132cb1 | 2021-08-09 17:54:47 +0100 | [diff] [blame] | 161 | fi |
Zelalem | 219df41 | 2020-05-17 19:21:20 -0500 | [diff] [blame] | 162 | |
Fathi Boudra | 422bf77 | 2019-12-02 11:10:16 +0200 | [diff] [blame] | 163 | # Check error count |
| 164 | |
| 165 | if [ "$ERROR_COUNT" != 0 ] || [ "$WARNING_COUNT" != 0 ]; then |
| 166 | echo "Some static checks have failed." |
| 167 | fi |
| 168 | |
| 169 | if [ "$ERROR_COUNT" != 0 ]; then |
| 170 | exit 1 |
| 171 | fi |
| 172 | |
| 173 | exit 0 |