Jayanth Dodderi Chidanand | 5132cb1 | 2021-08-09 17:54:47 +0100 | [diff] [blame] | 1 | #!/usr/bin/env bash |
| 2 | # |
Jayanth Dodderi Chidanand | 7f24bd2 | 2022-02-06 12:50:43 +0000 | [diff] [blame] | 3 | # Copyright (c) 2022 Arm Limited. All rights reserved. |
Jayanth Dodderi Chidanand | 5132cb1 | 2021-08-09 17:54:47 +0100 | [diff] [blame] | 4 | # |
| 5 | # SPDX-License-Identifier: BSD-3-Clause |
| 6 | # |
| 7 | |
| 8 | # static-checks-detect-newly-added-files.sh |
| 9 | # This script aims at identifying the newly added source files |
| 10 | # between the commits. |
| 11 | # It runs on every TF-A patch and detects the new files and updates |
| 12 | # the patch contibutor to include them for Coverity Scan analysis. |
| 13 | |
Paul Sokolovsky | 3304a3d | 2024-06-19 13:34:24 +0300 | [diff] [blame] | 14 | this_dir="$(readlink -f "$(dirname "$0")")" |
| 15 | . $this_dir/common.sh |
| 16 | |
| 17 | |
Jayanth Dodderi Chidanand | 5132cb1 | 2021-08-09 17:54:47 +0100 | [diff] [blame] | 18 | LOG_FILE=$(mktemp -t files-detection-check.XXXX) |
| 19 | TFA_PATCH_NEWFILES_LIST=$(mktemp -t tfa-patch-newfiles-list.XXXX) |
| 20 | EXIT_VALUE=0 |
Jayanth Dodderi Chidanand | 718b37b | 2021-09-14 01:34:02 +0100 | [diff] [blame] | 21 | DOC_URL="https://trustedfirmware-a.readthedocs.io/en/latest/process/contributing.html\ |
| 22 | #add-build-configurations" |
Jayanth Dodderi Chidanand | 5132cb1 | 2021-08-09 17:54:47 +0100 | [diff] [blame] | 23 | |
| 24 | # Function : file_updation_report |
| 25 | # Description : To update the inclusion of files listed in the temp file |
| 26 | # (tfa-patch-newfiles-list.XXXX) for Coverity Scan Analysis. |
| 27 | # Return : newly added source files,are captured onto the error log |
| 28 | # and the Error status is printed. |
| 29 | function file_updation_report( ) |
| 30 | { |
Jayanth Dodderi Chidanand | 718b37b | 2021-09-14 01:34:02 +0100 | [diff] [blame] | 31 | echo "===============================================================================" |
| 32 | echo >> "$LOG_FILE" |
| 33 | echo "New source files have been identified in your patch.." | tee -a "$LOG_FILE" |
Jayanth Dodderi Chidanand | 5132cb1 | 2021-08-09 17:54:47 +0100 | [diff] [blame] | 34 | # Iterating through the patch filenames and logging them onto error report. |
| 35 | while read filename |
| 36 | do |
Jayanth Dodderi Chidanand | 718b37b | 2021-09-14 01:34:02 +0100 | [diff] [blame] | 37 | echo "$filename" | tee -a "$LOG_FILE" |
Jayanth Dodderi Chidanand | 5132cb1 | 2021-08-09 17:54:47 +0100 | [diff] [blame] | 38 | done < "$TFA_PATCH_NEWFILES_LIST" |
| 39 | |
Jayanth Dodderi Chidanand | 718b37b | 2021-09-14 01:34:02 +0100 | [diff] [blame] | 40 | echo | tee -a "$LOG_FILE" |
Jayanth Dodderi Chidanand | 7f24bd2 | 2022-02-06 12:50:43 +0000 | [diff] [blame] | 41 | echo -e "1. Kindly ensure these newly added source files are covered by : \n\ |
| 42 | a. Coverity scan analysis by adding them to \"tf-cov-make\" build script with \ |
| 43 | the appropriate build configurations. \n\ |
| 44 | b. Built as part of one of the platform configurations present in \"tf-l1-build-plat\" \ |
| 45 | test group." | tee -a "$LOG_FILE" |
Jayanth Dodderi Chidanand | 5132cb1 | 2021-08-09 17:54:47 +0100 | [diff] [blame] | 46 | |
Jayanth Dodderi Chidanand | 718b37b | 2021-09-14 01:34:02 +0100 | [diff] [blame] | 47 | echo | tee -a "$LOG_FILE" |
Jayanth Dodderi Chidanand | 7f24bd2 | 2022-02-06 12:50:43 +0000 | [diff] [blame] | 48 | echo -e " Please refer to the tf-a documentation for more detailed explanation. \n\ |
| 49 | \"$DOC_URL\"" | tee -a "$LOG_FILE" |
| 50 | |
| 51 | echo | tee -a "$LOG_FILE" |
| 52 | echo -e "2. Please ignore if your files are already updated. Further the Code Maintainer will \n\ |
Jayanth Dodderi Chidanand | 718b37b | 2021-09-14 01:34:02 +0100 | [diff] [blame] | 53 | resolve the issue by taking appropriate action." | tee -a "$LOG_FILE" |
| 54 | echo "===============================================================================" |
Jayanth Dodderi Chidanand | 5132cb1 | 2021-08-09 17:54:47 +0100 | [diff] [blame] | 55 | |
| 56 | EXIT_VALUE=1 |
| 57 | } |
| 58 | |
Jayanth Dodderi Chidanand | df2faca | 2021-09-08 18:17:22 +0100 | [diff] [blame] | 59 | # Detecting source files not analysed by tf-coverity-job in the latest patch. |
Jayanth Dodderi Chidanand | 5132cb1 | 2021-08-09 17:54:47 +0100 | [diff] [blame] | 60 | echo "# Check to detect whether newly added files are analysed by Coverity in the patch" |
| 61 | TEST_CASE="Newly added files detection check for Coverity Scan analysis on patch(es)" |
| 62 | # Extracting newly added source files added between commits. |
Paul Sokolovsky | 3304a3d | 2024-06-19 13:34:24 +0300 | [diff] [blame] | 63 | git diff $(get_merge_base)..HEAD --name-only --diff-filter=A "*.c" &> "$TFA_PATCH_NEWFILES_LIST" |
Jayanth Dodderi Chidanand | 5132cb1 | 2021-08-09 17:54:47 +0100 | [diff] [blame] | 64 | if [ -s "$TFA_PATCH_NEWFILES_LIST" ] |
| 65 | then |
| 66 | file_updation_report |
| 67 | fi |
| 68 | |
| 69 | echo >> "$LOG_TEST_FILENAME" |
| 70 | echo "****** $TEST_CASE ******" >> "$LOG_TEST_FILENAME" |
| 71 | echo >> "$LOG_TEST_FILENAME" |
| 72 | |
| 73 | if [[ "$EXIT_VALUE" == 0 ]]; then |
| 74 | echo "Result : SUCCESS" >> "$LOG_TEST_FILENAME" |
| 75 | else |
| 76 | echo "Result : FAILURE" >> "$LOG_TEST_FILENAME" |
| 77 | fi |
| 78 | |
| 79 | # Printing the script output to show the warnings. |
| 80 | echo >> "$LOG_TEST_FILENAME" |
| 81 | cat "$LOG_FILE" >> "$LOG_TEST_FILENAME" |
| 82 | echo >> "$LOG_TEST_FILENAME" |
| 83 | |
| 84 | #Deleting temporary files |
| 85 | rm -f "$LOG_FILE" |
| 86 | rm -f "$TFA_PATCH_NEWFILES_LIST" |
| 87 | |
| 88 | exit "$EXIT_VALUE" |