Merge lp:~rodsmith/maas-cert-server/maniacs-tweaks into lp:~hardware-certification/maas-cert-server/trunk

Proposed by Rod Smith
Status: Merged
Approved by: Jeff Lane 
Approved revision: 71
Merged at revision: 71
Proposed branch: lp:~rodsmith/maas-cert-server/maniacs-tweaks
Merge into: lp:~hardware-certification/maas-cert-server/trunk
Diff against target: 45 lines (+10/-3)
2 files modified
debian/changelog (+6/-0)
usr/sbin/maniacs-setup (+4/-3)
To merge this branch: bzr merge lp:~rodsmith/maas-cert-server/maniacs-tweaks
Reviewer Review Type Date Requested Status
Jeff Lane  Approve
Review via email: mp+312798@code.launchpad.net

Description of the change

Some simple formatting changes I noticed would be desirable when doing a MANIACS revision.

To post a comment you must log in.
Revision history for this message
Jeff Lane  (bladernr) wrote :

+1

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/changelog'
2--- debian/changelog 2016-11-17 18:41:49 +0000
3+++ debian/changelog 2016-12-08 11:07:12 +0000
4@@ -1,3 +1,9 @@
5+maas-cert-server (0.2.28-0ppa1) UNRELEASED; urgency=medium
6+
7+ * Minor changes to maniacs-setup to improve prompt formatting.
8+
9+ -- Rod Smith <rod.smith@canonical.com> Thu, 08 Dec 2016 11:56:45 +0100
10+
11 maas-cert-server (0.2.27-0ppa1) xenial; urgency=medium
12
13 * Adjust maniacs-setup to download less stuff when pulling down cloud
14
15=== modified file 'usr/sbin/maniacs-setup'
16--- usr/sbin/maniacs-setup 2016-11-17 18:41:49 +0000
17+++ usr/sbin/maniacs-setup 2016-12-08 11:07:12 +0000
18@@ -529,7 +529,8 @@
19 fi
20 echo "* To defer this task, respond 'N' to the following question."
21 echo "*"
22- get_yn "* Do you want to set up a local cloud image mirror for the virtualization tests" "Y"
23+ echo "* Do you want to set up a local cloud image mirror for the virtualization"
24+ get_yn "* tests" "Y"
25 if [ $YN = "Y" ] ; then
26 if [ ! -d $CLOUD_MIRROR ] ; then
27 echo "* Cloud Mirror does not exist. Creating."
28@@ -541,7 +542,7 @@
29 if [ "$release" == `distro-info --lts` ] ; then
30 default_response="Y"
31 fi
32- get_yn "* Do you want to get images for $release release?" "$default_response"
33+ get_yn "* Do you want to get images for $release release" "$default_response"
34 if [ $YN = "Y" ]; then
35 get_these="$get_these $release"
36 fi
37@@ -552,7 +553,7 @@
38 if [ "$arch" == "amd64" ] ; then
39 default_response="Y"
40 fi
41- get_yn "* Do you want to get images for $arch architecture?" "$default_response"
42+ get_yn "* Do you want to get images for $arch architecture" "$default_response"
43 if [ $YN = "Y" ]; then
44 get_arches="$get_arches $arch"
45 fi

Subscribers

People subscribed via source and target branches

to status/vote changes: