From bce238d3927c48e7921643b7fec089f312491c09 Mon Sep 17 00:00:00 2001 From: Lucas Gravley <29484535+admiralAwkbar@users.noreply.github.com> Date: Mon, 24 Aug 2020 11:11:10 -0500 Subject: [PATCH 1/2] adding some spaget --- lib/buildFileList.sh | 168 +++++++++++++++++++++++++++++-------------- 1 file changed, 115 insertions(+), 53 deletions(-) diff --git a/lib/buildFileList.sh b/lib/buildFileList.sh index 53ea40b0..db10895d 100755 --- a/lib/buildFileList.sh +++ b/lib/buildFileList.sh @@ -190,15 +190,22 @@ function BuildFileList() { ###################### # Get the BASH files # ###################### - elif [ "${FILE_TYPE}" == "sh" ]; then - ################################ - # Append the file to the array # - ################################ - FILE_ARRAY_BASH+=("${FILE}") - ########################################################## - # Set the READ_ONLY_CHANGE_FLAG since this could be exec # - ########################################################## - READ_ONLY_CHANGE_FLAG=1 + elif [ "${FILE_TYPE}" == "sh" ] || [ "${FILE_TYPE}" == "bash" ] || + [ "${FILE_TYPE}" == "dash" ] || [ "${FILE_TYPE}" == "ksh" ]; then + # Need to check if its a zsh file as we cannot parse it + if CheckZsh "${FILE}"; then + warn "Found [zsh] script" + info "ShellCheck does NOT currently support zsh, skipping file" + else + ################################ + # Append the file to the array # + ################################ + FILE_ARRAY_BASH+=("${FILE}") + ########################################################## + # Set the READ_ONLY_CHANGE_FLAG since this could be exec # + ########################################################## + READ_ONLY_CHANGE_FLAG=1 + fi ###################### # Get the PERL files # ###################### @@ -506,50 +513,7 @@ function BuildFileList() { ############################################## # Use file to see if we can parse what it is # ############################################## - GET_FILE_TYPE_CMD=$(file "${FILE}" 2>&1) - - ################# - # Check if bash # - ################# - if [[ ${GET_FILE_TYPE_CMD} == *"Bourne-Again shell script"* ]]; then - ####################### - # It is a bash script # - ####################### - warn "Found bash script without extension:[.sh]" - info "Please update file with proper extensions." - ################################ - # Append the file to the array # - ################################ - FILE_ARRAY_BASH+=("${FILE}") - ########################################################## - # Set the READ_ONLY_CHANGE_FLAG since this could be exec # - ########################################################## - READ_ONLY_CHANGE_FLAG=1 - elif [[ ${GET_FILE_TYPE_CMD} == *"Ruby script"* ]]; then - ####################### - # It is a Ruby script # - ####################### - warn "Found ruby script without extension:[.rb]" - info "Please update file with proper extensions." - ################################ - # Append the file to the array # - ################################ - FILE_ARRAY_RUBY+=("${FILE}") - ########################################################## - # Set the READ_ONLY_CHANGE_FLAG since this could be exec # - ########################################################## - READ_ONLY_CHANGE_FLAG=1 - else - ############################ - # Extension was not found! # - ############################ - warn "Failed to get filetype for:[${FILE}]!" - ########################################################## - # Set the READ_ONLY_CHANGE_FLAG since this could be exec # - ########################################################## - READ_ONLY_CHANGE_FLAG=1 - fi - fi + CheckFileType "${FILE}" done export READ_ONLY_CHANGE_FLAG # Workaround SC2034 @@ -579,3 +543,101 @@ function BuildFileList() { info "----------------------------------------------" info "Successfully gathered list of files..." } +################################################################################ +#### Function CheckFileType #################################################### +function CheckFileType() { + # Need to run the file through the 'file' exec to help determine + # The type of file being parsed + + ################ + # Pull in Vars # + ################ + FILE="$1" + + ################## + # Check the file # + ################## + GET_FILE_TYPE_CMD=$(file "${FILE}" 2>&1) + + ################# + # Check if bash # + ################# + if [[ ${GET_FILE_TYPE_CMD} == *"Bourne-Again shell script"* ]]; then + ####################### + # It is a bash script # + ####################### + warn "Found bash script without extension:[.sh]" + info "Please update file with proper extensions." + ################################ + # Append the file to the array # + ################################ + FILE_ARRAY_BASH+=("${FILE}") + ########################################################## + # Set the READ_ONLY_CHANGE_FLAG since this could be exec # + ########################################################## + READ_ONLY_CHANGE_FLAG=1 + elif [[ ${GET_FILE_TYPE_CMD} == *"Ruby script"* ]]; then + ####################### + # It is a Ruby script # + ####################### + warn "Found ruby script without extension:[.rb]" + info "Please update file with proper extensions." + ################################ + # Append the file to the array # + ################################ + FILE_ARRAY_RUBY+=("${FILE}") + ########################################################## + # Set the READ_ONLY_CHANGE_FLAG since this could be exec # + ########################################################## + READ_ONLY_CHANGE_FLAG=1 + elif [[ ${GET_FILE_TYPE_CMD} == *"zsh script"* ]]; then + ###################### + # It is a ZSH script # + ###################### + warn "Found [zsh] script" + info "ShellCheck does NOT currently support zsh, skipping file" + else + ############################ + # Extension was not found! # + ############################ + warn "Failed to get filetype for:[${FILE}]!" + ########################################################## + # Set the READ_ONLY_CHANGE_FLAG since this could be exec # + ########################################################## + READ_ONLY_CHANGE_FLAG=1 + fi +} +################################################################################ +#### Function CheckZsh ######################################################### +function CheckZsh() { + # Spagetti code to make sure were properly excluding zsh + # until we get a proper linter + + ################ + # Pull in Vars # + ################ + FILE="$1" + + ################## + # Check the file # + ################## + GET_FILE_TYPE_CMD=$(file "${FILE}" 2>&1) + + if [[ ${GET_FILE_TYPE_CMD} == *"zsh script"* ]]; then + ###################### + # It is a ZSH script # + ###################### + warn "Found [zsh] script" + info "ShellCheck does NOT currently support zsh, skipping file" + ################################################### + # We found zsh file and need to return with a hit # + ################################################### + return 0 + else + ################## + # Not a zsh file # + ################## + return 1 + fi + +} From 6d7c89b7d756583d87c26f869c664ca0f7dcd7e4 Mon Sep 17 00:00:00 2001 From: Lucas Gravley <29484535+admiralAwkbar@users.noreply.github.com> Date: Mon, 24 Aug 2020 11:16:22 -0500 Subject: [PATCH 2/2] fix struct --- lib/buildFileList.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/buildFileList.sh b/lib/buildFileList.sh index db10895d..c3cf58cd 100755 --- a/lib/buildFileList.sh +++ b/lib/buildFileList.sh @@ -514,6 +514,7 @@ function BuildFileList() { # Use file to see if we can parse what it is # ############################################## CheckFileType "${FILE}" + fi done export READ_ONLY_CHANGE_FLAG # Workaround SC2034 @@ -639,5 +640,4 @@ function CheckZsh() { ################## return 1 fi - }