release-version: add an extra sanity check

ctrueden · ctrueden · commit 29fd15fa723f · 2022-08-18T13:48:13.000-05:00
In theory, the property extraction will exit non-zero whenever there is
an error, so this new check should never be triggered. But I'm paranoid,
so just in case, let's check the projectDetails result string, too.
diff --git a/release-version.sh b/release-version.sh
@@ -116,6 +116,8 @@ echoArg='${project.version}:${license.licenseName}:${project.parent.groupId}:${p
 projectDetails=$(mvn -N -Dexec.executable=echo -Dexec.args="$echoArg" exec:exec -q)
 test $? -eq 0 || projectDetails=$(mvn -U -N -Dexec.executable=echo -Dexec.args="$echoArg" exec:exec -q)
 test $? -eq 0 || die "Could not extract version from pom.xml. Error follows:\n$projectDetails"
+echo "$projectDetails" | grep -Fqv '[ERROR]' ||
+	die "Error extracting version from pom.xml. Error follows:\n$projectDetails"
 currentVersion=${projectDetails%%:*}
 projectDetails=${projectDetails#*:}
 licenseName=${projectDetails%%:*}
