launchpad-reviewers team mailing list archive

[Diogo Matsubara <matsubara@xxxxxxxxxxxxx>]

Diogo Matsubara has proposed merging lp:~matsubara/maas/fix-maas-cc-sed-precise-sru into lp:~maas-maintainers/maas/packaging.precise.sru.

Requested reviews:
  MAAS Maintainers (maas-maintainers)
Related bugs:
  Bug #1167660 in maas (Ubuntu): "maas-cluster-controller.postinst failing with an unterminated sed error"
  https://bugs.launchpad.net/ubuntu/+source/maas/+bug/1167660

For more details, see:
https://code.launchpad.net/~matsubara/maas/fix-maas-cc-sed-precise-sru/+merge/158352

Fix sed quoting in maas-cluster-controller.postinst

-- 
https://code.launchpad.net/~matsubara/maas/fix-maas-cc-sed-precise-sru/+merge/158352
Your team MAAS Maintainers is requested to review the proposed merge of lp:~matsubara/maas/fix-maas-cc-sed-precise-sru into lp:~maas-maintainers/maas/packaging.precise.sru.

=== modified file 'debian/maas-cluster-controller.postinst'
--- debian/maas-cluster-controller.postinst	2013-03-20 18:12:30 +0000
+++ debian/maas-cluster-controller.postinst	2013-04-11 12:42:23 +0000
@@ -83,7 +83,7 @@
     # Get the MAAS_URL on configure/reconfigure and write it to the conf files.
     db_get maas-cluster-controller/maas-url || true
     if [ -n "$RET" ]; then
-        sed -i "s|MAAS_URL=.*|MAAS_URL="$RET"|" /etc/maas/maas_cluster.conf
+        sed -i "s|MAAS_URL=.*|MAAS_URL='$RET'|" /etc/maas/maas_cluster.conf
         # Extract the hostname part.
         HOSTPART=$(echo $RET|awk '{ split($0,array,"/")} END{print array[3] }')
         # And substitute it in-place in pserv.yaml on an indented, non-commented