Merge pull request #1064 from erikwilson/fix-module-warnings

contrib/util/check-config.sh

@@ -19,6 +19,7 @@ possibleConfigs="
 "
 binDir=$(dirname "$0")
 configFormat=gz
+isError=0
 
 if [ $# -gt 0 ]; then
   CONFIG="$1"
@@ -108,7 +109,7 @@ check_flag() {
   elif is_set_as_module "$1"; then
     wrap_good "CONFIG_$1" 'enabled (as module)'
   else
-    if [ "$IS_ERROR" = 1 ]; then
+    if [ $isError -eq 1 ]; then
       wrap_bad "CONFIG_$1" 'missing'
     else
       wrap_warn "CONFIG_$1" 'missing'
@@ -269,7 +270,7 @@ echo
 
 SUDO=
 [ $(id -u) -ne 0 ] && SUDO=sudo
-lsmod | grep -q configs || $SUDO modprobe configs || true
+lsmod | grep -q configs || $SUDO modprobe configs >/dev/null 2>&1 || true
 
 if [ -z "$CONFIG" ]; then
   for tryConfig in ${possibleConfigs}; do
@@ -341,7 +342,8 @@ flags="
   IP_NF_NAT NF_NAT NF_NAT_NEEDED
   POSIX_MQUEUE
 "
-IS_ERROR=1 check_flags $flags
+isError=1 check_flags $flags && isError=0
+
 if [ "$kernelMajor" -lt 4 ] || ( [ "$kernelMajor" -eq 4 ] && [ "$kernelMinor" -lt 8 ] ); then
   check_flags DEVPTS_MULTIPLE_INSTANCES
 fi