Lars Schneider <larsxschneider@xxxxxxxxx> writes: > On 02 May 2016, at 22:45, Junio C Hamano <gitster@xxxxxxxxx> wrote: > >> larsxschneider@xxxxxxxxx writes: >> >>> +set -e >>> + >>> +LINKS=$(grep --recursive --only-matching --no-filename --perl-regexp \ >>> + '(?<=linkgit:).*?(?=\[\d+\])' Documentation/* \ >>> + | sort -u \ >>> +) >>> + >>> +for LINK in $LINKS; do >>> + echo "Checking linkgit:$LINK..." >>> + test -s Documentation/$LINK.txt >>> +done >> >> Please separate the above link check out of this step and do so >> separately after the move of test body to a separate script >> settles. > > OK. I also wonder if the link check should rather go to the > "check-docs" Makefile target? That sounds like a good direction. Which in turn means that people on all platforms are welcome to run it, which in turn means that the script must be even more portable, with avoiding GNUism and bash-isms etc. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html