Browse Source

tap_tool_dispatcher: use .tap suffix for output files

See https://issues.jenkins-ci.org/browse/JENKINS-15813 ->

| If the file type is considered as binary by Jenkins webserver
| then the TAP plugin doesn't consider this file for inclusion in
| TAP reports. This can be quite confusing and if you're not aware
| of this issue it can easily hide problems.

So don't use just "$filename" but always "$filename.tap" instead.
merge-requests/109/head
Michael Prokop 10 years ago
parent
commit
b56f16b1ae
  1. 4
      tap/tap_tool_dispatcher

4
tap/tap_tool_dispatcher

@ -24,12 +24,12 @@ mkdir -p "${REPORTS_DIRECTORY}"/shell
# run perlcritic_tap
echo "*** perlcritic_tap ***"
find source -type f ! -path '*.svn*' ! -path '*.git/*' -prune -exec \
sh -c 'input="$1"; output=$(echo $input | sed -e s,/,_,g); /usr/bin/perlcritic_tap "$input" > "${REPORTS_DIRECTORY}/perl/${output}"' '{}' '{}' ';'
sh -c 'input="$1"; output=$(echo $input | sed -e s,/,_,g); /usr/bin/perlcritic_tap "$input" > "${REPORTS_DIRECTORY}/perl/${output}".tap' '{}' '{}' ';'
# run checkbashism_tap
echo "*** checkbashism_tap ***"
find source -type f ! -path '*.svn*' ! -path '*.git/*' -prune -exec \
sh -c 'input="$1"; output=$(echo $input | sed -e s,/,_,g); /usr/bin/checkbashism_tap "$input" > "${REPORTS_DIRECTORY}/shell/${output}"' '{}' '{}' ';'
sh -c 'input="$1"; output=$(echo $input | sed -e s,/,_,g); /usr/bin/checkbashism_tap "$input" > "${REPORTS_DIRECTORY}/shell/${output}".tap' '{}' '{}' ';'
# get rid of empty files
find "${REPORTS_DIRECTORY}" -type f -empty -exec rm {} +

Loading…
Cancel
Save