X-Git-Url: https://diplodocus.org/git/nmh/blobdiff_plain/f2ac699d39bce1406ba54ff6cf9046c954361411..912755a7e0ad3d94d084798dd60bb7397abf3931:/tools/showbuildenv

diff --git a/tools/showbuildenv b/tools/showbuildenv
index 5eec6ee8..9db0f3e2 100755
--- a/tools/showbuildenv
+++ b/tools/showbuildenv
@@ -5,13 +5,14 @@
 # build environment on machines they can't login on.  It also provides
 # a history in the build logs that might be useful in catching
 # regressions.
-# 
+#
 # This script should *only* report basic information about the build
 # environment, and only that necessary to reproduce the build
 # environment.
 #
-# FIXME: Document meaning of script's exit status, if any.
-# It jumps through hoops with `|| true' despite not using `set -e'.
+# The script's exit status must be zero if all went acceptably well as
+# buildbot checks for this and marks the `environment' stage as failed
+# otherwise.
 
 offset () {
     sed -e '/./s/^/    /' "$@"
@@ -28,7 +29,7 @@ header () {
 # also use that to placate broken logic that use its prescence as a
 # use-RPM test.
 for f in \
-    /etc/os-release /etc/arch-release /etc/debian_version \
+    /etc/os-release /etc/arch-release /etc/centos-release /etc/debian_version \
     /etc/fedora-release /etc/gentoo-release /etc/redhat-release \
     /etc/lsb-release \
     ; do
@@ -62,7 +63,8 @@ case ${os} in
 	header Compilers:
 	cc -v 2>&1 | offset
 
-  	header Installed packages:
+	header Installed packages:
+        #### pkg(7) or pkg_info(1) might not always be available.
 	(pkg info 2>/dev/null | offset) || true
 	(pkg_info 2>/dev/null | offset) || true
 	;;