launchpad-reviewers team mailing list archive

[Diogo Matsubara <matsubara@xxxxxxxxxxxxx>]

Diogo Matsubara has proposed merging lp:~matsubara/maas/fix-MAAS_URL-quotes into lp:~maas-maintainers/maas/packaging.

Commit message:
Fix the way MAAS_URL is quoted.

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)
Related bugs:
  Bug #1172303 in MAAS: "MAAS_URL on maas_cluster.conf is quoted on quantal but not quoted on raring"
  https://bugs.launchpad.net/maas/+bug/1172303

For more details, see:
https://code.launchpad.net/~matsubara/maas/fix-MAAS_URL-quotes/+merge/160722

Fix the way MAAS_URL is quoted.
-- 
https://code.launchpad.net/~matsubara/maas/fix-MAAS_URL-quotes/+merge/160722
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~matsubara/maas/fix-MAAS_URL-quotes into lp:~maas-maintainers/maas/packaging.

=== modified file 'debian/maas-cluster-controller.postinst'
--- debian/maas-cluster-controller.postinst	2013-04-11 01:38:28 +0000
+++ debian/maas-cluster-controller.postinst	2013-04-24 17:38:27 +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