Check completeness of help output in CI

This commit adds a new script to run post-build that assesses the
completeness of --help for all binaries except for goto-cc.

Author: tautschnig

.github/workflows/pull-request-checks.yaml

@@ -55,6 +55,8 @@ jobs:
           make -C jbmc/unit -j2 CXX="ccache g++" LIBS="$PWD/riss.git/release/lib/libriss-coprocessor.a -lpthread" IPASIR=$PWD/riss.git/riss
       - name: Print ccache stats
         run: ccache -s
+      - name: Checking completeness of help output
+        run: scripts/check_help.sh
       - name: Run unit tests
         run: |
           make -C unit test IPASIR=$PWD/riss.git/riss
@@ -233,6 +235,8 @@ jobs:
         run: cd build; ninja -j2
       - name: Print ccache stats
         run: ccache -s
+      - name: Checking completeness of help output
+        run: scripts/check_help.sh build/bin
       - name: Check if package building works
         run: |
           cd build

scripts/check_help.sh

@@ -0,0 +1,82 @@
+#!/bin/bash
+
+set -e
+
+# if a command-line argument is provided, use it as a path to built binaries
+# (CMake-style build); otherwise assume we use Makefile-based in-tree build
+if [ $# -eq 1 ] ; then
+  bin_dir=$(cd $1 ; pwd)
+else
+  unset bin_dir
+fi
+echo $bin_dir
+
+# make sure we execute the remainder in the directory containing this script
+cd `dirname $0`
+
+missing_options=0
+
+# we don't check goto-cc for now
+# we omit crangler for it doesn't take options other than help or a file name
+for t in  \
+  ../jbmc/src/janalyzer \
+  ../jbmc/src/jbmc \
+  ../jbmc/src/jdiff \
+  ../src/cbmc \
+  ../src/goto-analyzer \
+  ../src/goto-diff \
+  ../src/goto-harness \
+  ../src/goto-instrument \
+  ../src/memory-analyzer \
+  ../src/symtab2gb \
+; do
+  tool_name=$(basename $t)
+  opt_name=$(echo $tool_name | tr 'a-z-' 'A-Z_')
+	echo "Extracting the raw list of parameters from $tool_name"
+  g++ -E -dM -std=c++11 -I../src -I../jbmc/src $t/*_parse_options.cpp -o macros.c
+  # goto-analyzer partly uses the spelling "analyser" within the code base
+  echo ${opt_name}_OPTIONS | sed 's/GOTO_ANALYZER/GOTO_ANALYSER/' >> macros.c
+  rawstring="`gcc -E -P -w macros.c` \"?h(help)\""
+  rm macros.c
+
+  # now the main bit, convert from raw format to a proper list of switches
+  cleanstring=`(
+    # extract 2-hyphen switches, such as --foo
+    # grep for '(foo)' expressions, and then use sed to remove parantheses and
+    # put '-' at the start (we accept both --X and -X)
+    (echo $rawstring | grep -o "([^)]*)" | sed "s/^.\(.*\).$/-\1/") ;
+    # extract 1-hyphen switches, such as -F
+    # use sed to remove all (foo) expressions, then you're left with switches
+    # and ':', so grep the colons out and then use sed to include the '-'
+    (echo $rawstring | sed "s/([^)]*)//g" | grep -o "[a-zA-Z0-9?]" | sed "s/\(.*\)/-\1/")
+  ) | sed 's/" "//g'`
+
+  if [ "x$bin_dir" = "x" ] ; then
+    if [ ! -x $t/$tool_name ] ; then
+      echo "$t/$tool_name is not an executable"
+      exit 1
+    fi
+    $t/$tool_name --help > help_string
+  else
+    if [ ! -x $bin_dir/$tool_name ] ; then
+      echo "$bin_dir/$tool_name is not an executable"
+      exit 1
+    fi
+    $bin_dir/$tool_name --help > help_string
+  fi
+  for opt in $cleanstring ; do
+    if ! grep -q -- $opt help_string ; then
+      echo "Option $opt of $tool_name is undocumented"
+      missing_options=1
+    fi
+  done
+  rm help_string
+done
+
+if [ $missing_options -eq 1 ] ; then
+  echo "Undocumented options found"
+  exit 1
+else
+  echo "All options are documented"
+  exit 0
+fi