Fix check-supported-tools-tables check

sed wasn't using -E, so '|' wasn't being handled properly. Seems likely
that's sed-implementation specific, so now it runs through docker's sed
to support portability.
This commit is contained in:
Kevin Clark
2020-11-27 20:32:51 -08:00
parent 1365dce921
commit f72e60c12a
2 changed files with 5 additions and 3 deletions

View File

@@ -22,7 +22,7 @@ done < <(
grep '^\*\|^ *\*' doc/ale-supported-languages-and-tools.txt \ grep '^\*\|^ *\*' doc/ale-supported-languages-and-tools.txt \
| sed -e '1,2d' \ | sed -e '1,2d' \
| sed 's/^\* */!/' \ | sed 's/^\* */!/' \
| sed 's/^ *\* *\|!!\|\^\|(.*)\|`//g' \ | sed -E 's/^ *\* *|!!|\^|(.*)|`//g' \
| sed 's/ *$//' | sed 's/ *$//'
) > "$doc_file" ) > "$doc_file"
@@ -36,7 +36,7 @@ while read -r; do
done < <( done < <(
grep '^\*\|^ *\*' supported-tools.md \ grep '^\*\|^ *\*' supported-tools.md \
| sed 's/^\* */!/' \ | sed 's/^\* */!/' \
| sed 's/^ *\* *\|:floppy_disk:\|:warning:\|(.*)\|\[\|\].*\|-n flag//g' \ | sed -E 's/^ *\* *|:floppy_disk:|:warning:|\(.*\)|\[|\].*|-n flag//g' \
| sed 's/ *$//' | sed 's/ *$//'
) > "$readme_file" ) > "$readme_file"

View File

@@ -46,7 +46,9 @@ echo '========================================'
echo 'Differences follow:' echo 'Differences follow:'
echo echo
test/script/check-supported-tools-tables || exit_code=$? set -o pipefail
docker run "${docker_flags[@]}" test/script/check-supported-tools-tables || exit_code=$?
set +o pipefail
echo '========================================' echo '========================================'
echo 'Look for badly aligned doc tags' echo 'Look for badly aligned doc tags'