contrib: fix variable substitution

bd808 · bd808 · commit 1be38a26948a · 2016-07-06T22:36:19.000-06:00
diff --git a/contrib/add-tool.sh b/contrib/add-tool.sh
@@ -9,7 +9,7 @@ BASE_DN="dc=wmftest,dc=net"
 TOOL_BASE_DN="ou=servicegroups,${BASE_DN}"
 ADMIN_DN="cn=admin,${BASE_DN}"
 ADMIN_PASS="vagrant_admin"
-MEMBER_DN=${2:=$ADMIN_DN}
+MEMBER_DN=${2:-$ADMIN_DN}
 
 # Crappy random id generator
 NEW_GID=$(echo -n "${TOOL}" | sha1sum | awk '{print $1}' | tr '[:lower:]' '[:upper:]')
diff --git a/contrib/add-user.sh b/contrib/add-user.sh
@@ -6,7 +6,7 @@
 # Usage: add-user.sh SHELL_USER_NAME [WIKI_USER_NAME]
 
 NEW_UID=${1:?SHELL_USER_NAME required}
-NEW_CN=${2:=$NEW_UID}
+NEW_CN=${2:-$NEW_UID}
 BASE_DN="dc=wmftest,dc=net"
 USER_BASE_DN="ou=People,${BASE_DN}"
 ADMIN_DN="cn=admin,${BASE_DN}"
diff --git a/contrib/del-tool.sh b/contrib/del-tool.sh
@@ -0,0 +1,16 @@
+#!/usr/bin/env bash
+# Hack a tool into the testing LDAP server setup with MediaWiki-Vagrant's
+# role::striker.
+#
+# Usage: add-tool.sh NAME [DN_OF_MAINTAINER]
+
+TOOL=${1:?TOOL required}
+BASE_DN="dc=wmftest,dc=net"
+TOOL_BASE_DN="ou=servicegroups,${BASE_DN}"
+ADMIN_DN="cn=admin,${BASE_DN}"
+ADMIN_PASS="vagrant_admin"
+
+/usr/bin/ldapadd -x -D "${ADMIN_DN}" -w "${ADMIN_PASS}" <<LDIF
+dn: cn=tools.${TOOL},${TOOL_BASE_DN}
+changetype: delete
+LDIF
