Merge lp:~andreserl/maas/packaging_update_cluster into lp:~maas-maintainers/maas/packaging

Proposed by Andres Rodriguez
Status: Merged
Approved by: Raphaël Badin
Approved revision: no longer in the source branch.
Merged at revision: 121
Proposed branch: lp:~andreserl/maas/packaging_update_cluster
Merge into: lp:~maas-maintainers/maas/packaging
Diff against target: 32 lines (+5/-2)
2 files modified
debian/changelog (+2/-1)
debian/maas-cluster-controller.postinst (+3/-1)
To merge this branch: bzr merge lp:~andreserl/maas/packaging_update_cluster
Reviewer Review Type Date Requested Status
Raphaël Badin (community) Approve
Review via email: mp+128564@code.launchpad.net

Commit message

debian/maas-cluster-controller.postinst: Only set URL if not empty.

To post a comment you must log in.
Revision history for this message
Raphaël Badin (rvb) wrote :

Looks good.

review: Approve
Revision history for this message
MAAS Lander (maas-lander) wrote :

No commit message specified.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/changelog'
--- debian/changelog 2012-10-08 17:16:30 +0000
+++ debian/changelog 2012-10-08 18:39:20 +0000
@@ -1,4 +1,4 @@
1maas (0.1+bzr1199+dfsg-0ubuntu1) UNRELEASED; urgency=low1maas (0.1+bzr1223+dfsg-0ubuntu1~ppa1) quantal; urgency=low
22
3 * New upstream release. (LP: #1062518)3 * New upstream release. (LP: #1062518)
44
@@ -40,6 +40,7 @@
40 appropriate etc dir.40 appropriate etc dir.
41 * debian/maas-cluster-controller.config: Source debconf at the beginning of the41 * debian/maas-cluster-controller.config: Source debconf at the beginning of the
42 script. (LP: #1063857)42 script. (LP: #1063857)
43 * debian/maas-cluster-controller.postinst: Only set URL if not empty.
4344
44 [ Robie Basak ]45 [ Robie Basak ]
45 * Add maas-cluster-controller dependency on uuid-runtime, needed for uuidgen46 * Add maas-cluster-controller dependency on uuid-runtime, needed for uuidgen
4647
=== modified file 'debian/maas-cluster-controller.postinst'
--- debian/maas-cluster-controller.postinst 2012-10-02 02:29:02 +0000
+++ debian/maas-cluster-controller.postinst 2012-10-08 18:39:20 +0000
@@ -37,7 +37,9 @@
37if ([ "$1" = "configure" ] && [ -z "$2" ]) || [ "$1" = "reconfigure" ] || [ -n "$DEBCONF_RECONFIGURE" ]; then37if ([ "$1" = "configure" ] && [ -z "$2" ]) || [ "$1" = "reconfigure" ] || [ -n "$DEBCONF_RECONFIGURE" ]; then
38 # Get the MAAS_URL on configure/reconfigure and write it to the conf file.38 # Get the MAAS_URL on configure/reconfigure and write it to the conf file.
39 db_get maas-cluster-controller/maas-url || true39 db_get maas-cluster-controller/maas-url || true
40 sed -i "s|MAAS_URL=.*|MAAS_URL="$RET"|" /etc/maas/maas_cluster.conf40 if [ -n "$RET" ]; then
41 sed -i "s|MAAS_URL=.*|MAAS_URL="$RET"|" /etc/maas/maas_cluster.conf
42 fi
41fi43fi
4244
43#DEBHELPER#45#DEBHELPER#

Subscribers

People subscribed via source and target branches

to all changes: