MAAS

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:

Merge into:

Diff against target:

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

Medium

Fix Released

Link a bug report

Reviewer	Review Type	Date Requested	Status
Raphaël Badin (community)		2013-04-11	Approve on 2013-04-11
Review via email: mp+158352@code.launchpad.net

Fix sed quoting in maas-cluster-controller.postinst

Raphaël Badin (rvb) on 2013-04-11:

review: Approve

Raphaël Badin (rvb) wrote on 2013-04-11:

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

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk

People subscribed via source and target branches

to all changes:

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