Lars Schneider <larsxschnei...@gmail.com> writes: > On 02 May 2016, at 22:45, Junio C Hamano <gits...@pobox.com> wrote: > >> larsxschnei...@gmail.com 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 majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
