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 | |
Fathi Boudra | 422bf77 | 2019-12-02 11:10:16 +0200 | [diff] [blame] | 20 | # Initialize log file |
| 21 | |
| 22 | export LOG_TEST_FILENAME=$(pwd)/static-checks.log |
| 23 | |
| 24 | echo |
| 25 | echo "###### Static checks ######" |
| 26 | echo |
| 27 | |
| 28 | echo "###### Static checks ######" > "$LOG_TEST_FILENAME" |
| 29 | echo >> "$LOG_TEST_FILENAME" |
| 30 | |
Paul Sokolovsky | 422020c | 2024-06-18 23:21:07 +0300 | [diff] [blame] | 31 | echo "Patch series being checked:" >> "$LOG_TEST_FILENAME" |
| 32 | git log --oneline $(get_merge_base)..HEAD >> "$LOG_TEST_FILENAME" |
| 33 | echo >> "$LOG_TEST_FILENAME" |
| 34 | echo "Base branch reference commit:" >> "$LOG_TEST_FILENAME" |
| 35 | git log --oneline -1 $(get_merge_base) >> "$LOG_TEST_FILENAME" |
| 36 | |
| 37 | echo >> "$LOG_TEST_FILENAME" |
| 38 | |
Fathi Boudra | 422bf77 | 2019-12-02 11:10:16 +0200 | [diff] [blame] | 39 | # Reset error counters |
| 40 | |
| 41 | ERROR_COUNT=0 |
| 42 | WARNING_COUNT=0 |
| 43 | |
| 44 | # Ensure all the files contain a copyright |
| 45 | |
| 46 | echo 'Checking copyright in source files...' |
| 47 | echo |
| 48 | "$CI_ROOT"/script/static-checks/static-checks-check-copyright.sh . |
| 49 | if [ "$?" != 0 ]; then |
| 50 | echo "Copyright test: FAILURE" |
| 51 | ((ERROR_COUNT++)) |
| 52 | else |
| 53 | echo "Copyright test: PASS" |
| 54 | fi |
| 55 | echo |
| 56 | |
| 57 | # Check alphabetic order of headers included. |
| 58 | |
| 59 | if [ "$IS_CONTINUOUS_INTEGRATION" == 1 ]; then |
| 60 | "$CI_ROOT"/script/static-checks/static-checks-include-order.sh . patch |
| 61 | else |
| 62 | "$CI_ROOT"/script/static-checks/static-checks-include-order.sh . |
| 63 | fi |
| 64 | if [ "$?" != 0 ]; then |
| 65 | echo "Include order test: FAILURE" |
| 66 | ((WARNING_COUNT++)) |
| 67 | else |
| 68 | echo "Include order test: PASS" |
| 69 | fi |
| 70 | echo |
| 71 | |
| 72 | # Check line endings |
| 73 | |
Leonardo Sandoval | 239e8ac | 2020-07-27 15:14:59 -0500 | [diff] [blame] | 74 | if [ "$IS_CONTINUOUS_INTEGRATION" == 1 ]; then |
| 75 | "$CI_ROOT"/script/static-checks/static-checks-coding-style-line-endings.sh . patch |
| 76 | else |
| 77 | "$CI_ROOT"/script/static-checks/static-checks-coding-style-line-endings.sh |
| 78 | fi |
| 79 | |
Fathi Boudra | 422bf77 | 2019-12-02 11:10:16 +0200 | [diff] [blame] | 80 | if [ "$?" != 0 ]; then |
| 81 | echo "Line ending test: FAILURE" |
| 82 | ((ERROR_COUNT++)) |
| 83 | else |
| 84 | echo "Line ending test: PASS" |
| 85 | fi |
| 86 | echo |
| 87 | |
| 88 | # Check coding style |
| 89 | |
| 90 | echo 'Checking coding style compliance...' |
| 91 | echo |
| 92 | if [ "$IS_CONTINUOUS_INTEGRATION" == 1 ]; then |
| 93 | "$CI_ROOT"/script/static-checks/static-checks-coding-style.sh |
| 94 | else |
| 95 | "$CI_ROOT"/script/static-checks/static-checks-coding-style-entire-src-tree.sh |
| 96 | fi |
| 97 | if [ "$?" != 0 ]; then |
| 98 | echo "Coding style test: FAILURE" |
| 99 | ((ERROR_COUNT++)) |
| 100 | else |
| 101 | echo "Coding style test: PASS" |
| 102 | fi |
| 103 | echo |
| 104 | |
Zelalem | 219df41 | 2020-05-17 19:21:20 -0500 | [diff] [blame] | 105 | # Check for any Banned API usage |
| 106 | |
| 107 | echo 'Checking Banned API usage...' |
| 108 | echo |
| 109 | if [ "$IS_CONTINUOUS_INTEGRATION" == 1 ]; then |
| 110 | "$CI_ROOT"/script/static-checks/static-checks-banned-apis.sh . patch |
| 111 | else |
| 112 | "$CI_ROOT"/script/static-checks/static-checks-banned-apis.sh |
| 113 | fi |
| 114 | if [ "$?" != 0 ]; then |
| 115 | echo "Banned API check: FAILURE" |
| 116 | ((ERROR_COUNT++)) |
| 117 | else |
| 118 | echo "Banned API check: PASS" |
| 119 | fi |
| 120 | echo |
| 121 | |
Jayanth Dodderi Chidanand | 5132cb1 | 2021-08-09 17:54:47 +0100 | [diff] [blame] | 122 | # Check to ensure newly added source files are detected for Coverity Scan analysis |
| 123 | |
Jayanth Dodderi Chidanand | 253b339 | 2021-09-10 12:40:00 +0100 | [diff] [blame] | 124 | # Check to be executed only on trusted-firmware repository. |
Manish V Badarkhe | 8b846f4 | 2021-10-13 15:43:58 +0100 | [diff] [blame] | 125 | 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] | 126 | echo 'Checking whether the newly added source files are detected for Coverity Scan analysis...' |
| 127 | echo |
| 128 | "$CI_ROOT"/script/static-checks/static-checks-detect-newly-added-files.sh |
| 129 | if [ "$?" != 0 ]; then |
| 130 | echo "Files Detection check: FAILURE" |
| 131 | ((ERROR_COUNT++)) |
| 132 | else |
| 133 | echo "Files Detection check: PASS" |
| 134 | fi |
| 135 | echo |
Jayanth Dodderi Chidanand | 5132cb1 | 2021-08-09 17:54:47 +0100 | [diff] [blame] | 136 | fi |
Zelalem | 219df41 | 2020-05-17 19:21:20 -0500 | [diff] [blame] | 137 | |
Fathi Boudra | 422bf77 | 2019-12-02 11:10:16 +0200 | [diff] [blame] | 138 | # Check error count |
| 139 | |
| 140 | if [ "$ERROR_COUNT" != 0 ] || [ "$WARNING_COUNT" != 0 ]; then |
| 141 | echo "Some static checks have failed." |
| 142 | fi |
| 143 | |
| 144 | if [ "$ERROR_COUNT" != 0 ]; then |
| 145 | exit 1 |
| 146 | fi |
| 147 | |
| 148 | exit 0 |