Merge lp:~matsubara/maas/fix-maas-cc-sed-precise-sru into lp:~maas-maintainers/maas/packaging.precise.sru

Proposed by Diogo Matsubara on 2013-04-11
Status: Merged
Approved by: Diogo Matsubara on 2013-04-11
Approved revision: 170
Merged at revision: 170
Proposed branch: lp:~matsubara/maas/fix-maas-cc-sed-precise-sru
Merge into: lp:~maas-maintainers/maas/packaging.precise.sru
Diff against target: 12 lines (+1/-1)
1 file modified
debian/maas-cluster-controller.postinst (+1/-1)
To merge this branch: bzr merge lp:~matsubara/maas/fix-maas-cc-sed-precise-sru
Reviewer Review Type Date Requested Status
Raphaël Badin (community) 2013-04-11 Approve on 2013-04-11
Review via email:

Description of the change

Fix sed quoting in maas-cluster-controller.postinst

To post a comment you must log in.
Raphaël Badin (rvb) :
review: Approve
Raphaël Badin (rvb) wrote :

btw, IIRC there is no lander for this branch so you'll have to merge it manually.

Preview Diff

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


People subscribed via source and target branches