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

Proposed by Andres Rodriguez
Status: Merged
Approved by: Andres Rodriguez
Approved revision: no longer in the source branch.
Merged at revision: 22
Proposed branch: lp:~andreserl/maas/maas_packaging_updates
Merge into: lp:~maas-maintainers/maas/packaging
Diff against target: 238 lines (+33/-71)
12 files modified
debian/changelog (+8/-2)
debian/control (+1/-1)
debian/maas.config (+4/-1)
debian/maas.install (+2/-2)
debian/maas.links (+1/-1)
debian/maas.maas-pserv.upstart (+4/-4)
debian/maas.maas-txlongpoll.upstart (+2/-2)
debian/maas.postinst (+5/-5)
debian/patches/01-fix-database-settings.patch (+4/-4)
debian/po/POTFILES.in (+0/-1)
debian/po/templates.pot (+0/-46)
debian/rules (+2/-2)
To merge this branch: bzr merge lp:~andreserl/maas/maas_packaging_updates
Reviewer Review Type Date Requested Status
Francis J. Lacoste (community) Approve
Review via email: mp+97735@code.launchpad.net

Commit message

Various packaging improvements that include:
 * configure db only on configure/reconfigure
 * install maas_local_settings.py to etc/maas
 * Update references from MaaS to MAAS
 * Remove debian/po

To post a comment you must log in.
Revision history for this message
Francis J. Lacoste (flacoste) wrote :

Looks good.

Revision history for this message
Francis J. Lacoste (flacoste) :
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 2012-03-14 23:29:38 +0000
3+++ debian/changelog 2012-03-15 19:43:24 +0000
4@@ -1,4 +1,4 @@
5-maas (0.1+bzr283+dfsg-0ubuntu1) UNRELEASED; urgency=low
6+maas (0.1+bzr295+dfsg-0ubuntu1) precise; urgency=low
7
8 * debian/maas.upstart: Specify pidfile and conffile.
9 * debian/maas.postinst:
10@@ -14,8 +14,14 @@
11 * debian/maas.postinst: Touch logfile to set correct permissions (LP: #955400)
12 * debian/maas.logrotate: Add.
13 * debian/patches/03-txlongpoll-config.patch: Add default settings.
14+ * debian/maas.config: Only run dbconfig-common on configure/reconfigure.
15+ * Install maas_local_settings.py in /etc/maas and symlink to usr/share/maas
16+ * Update references from MaaS to MAAS.
17+ * Remove podebconf as templates no longer exist.
18+ - debian/po: Remove.
19+ - debian/control: Drop Build-Depends on po-debconf
20
21- -- Andres Rodriguez <andreserl@ubuntu.com> Wed, 14 Mar 2012 18:27:31 -0400
22+ -- Andres Rodriguez <andreserl@ubuntu.com> Thu, 15 Mar 2012 15:28:14 -0400
23
24 maas (0.1+bzr266+dfsg-0ubuntu1) precise; urgency=low
25
26
27=== modified file 'debian/control'
28--- debian/control 2012-03-14 23:29:38 +0000
29+++ debian/control 2012-03-15 19:43:24 +0000
30@@ -2,7 +2,7 @@
31 Section: net
32 Priority: optional
33 Maintainer: Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>
34-Build-Depends: debhelper (>= 8), python (>= 2.7), python-distribute, po-debconf
35+Build-Depends: debhelper (>= 8), python (>= 2.7), python-distribute
36 Standards-Version: 3.9.3
37 X-Python-Version: >= 2.7
38 Homepage: https://launchpad.net/maas
39
40=== modified file 'debian/maas.config'
41--- debian/maas.config 2012-03-13 18:16:33 +0000
42+++ debian/maas.config 2012-03-15 19:43:24 +0000
43@@ -5,8 +5,11 @@
44
45 # source dbconfig-common shell library, and call the hook function
46 if [ -f /usr/share/dbconfig-common/dpkg/config.pgsql ]; then
47+ . /usr/share/dbconfig-common/dpkg/config.pgsql
48+fi
49+
50+if ([ "$1" = "configure" ] && [ -z "$2" ]) || [ "$1" = "reconfigure" ] || [ -n "$DEBCONF_RECONFIGURE" ]; then
51 maas_db_pass=`pwgen`
52- . /usr/share/dbconfig-common/dpkg/config.pgsql
53 dbc_dbname="maasdb"
54 dbc_dbuser="maas"
55 dbc_dbpass="$maas_db_pass"
56
57=== modified file 'debian/maas.install'
58--- debian/maas.install 2012-03-14 23:29:38 +0000
59+++ debian/maas.install 2012-03-15 19:43:24 +0000
60@@ -10,8 +10,8 @@
61 # Install Apache Config
62 contrib/maas-http.conf etc/maas
63
64-# Install MaaS local settings
65-debian/tmp/usr/share/maas/maas_local_settings.py
66+# Install MAAS local settings
67+debian/tmp/etc/maas/maas_local_settings.py
68
69 # Install WSGI
70 contrib/wsgi.py usr/share/maas
71
72=== modified file 'debian/maas.links'
73--- debian/maas.links 2012-03-07 17:38:33 +0000
74+++ debian/maas.links 2012-03-15 19:43:24 +0000
75@@ -1,1 +1,1 @@
76-usr/share/maas/maas_local_settings.py etc/maas/maas_local_settings.py
77+etc/maas/maas_local_settings.py usr/share/maas/maas_local_settings.py
78
79=== modified file 'debian/maas.maas-pserv.upstart'
80--- debian/maas.maas-pserv.upstart 2012-03-14 23:29:38 +0000
81+++ debian/maas.maas-pserv.upstart 2012-03-15 19:43:24 +0000
82@@ -1,8 +1,8 @@
83-# maas - provisioning service
84+# maas-pserv - provisioning service
85 #
86-# MaaS Provisioning Service
87+# MAAS Provisioning Service
88
89-description "MaaS"
90+description "MAAS"
91 author "Andres Rodriguez <andres.rodriguez@canonical.com>"
92
93 start on filesystem and net-device-up
94@@ -11,4 +11,4 @@
95 respawn
96
97 # To add options to your daemon, edit the line below:
98-exec /usr/bin/twistd -n --pidfile=/run/maas-pserv.pid --logfile=/var/log/maas/maas-pserv.log maas-pserv --config-file=/etc/maas/pserv.yaml
99+exec /usr/bin/twistd -n --pidfile=/run/maas-pserv.pid --logfile=/dev/null maas-pserv --config-file=/etc/maas/pserv.yaml
100
101=== modified file 'debian/maas.maas-txlongpoll.upstart'
102--- debian/maas.maas-txlongpoll.upstart 2012-03-14 23:29:38 +0000
103+++ debian/maas.maas-txlongpoll.upstart 2012-03-15 19:43:24 +0000
104@@ -1,8 +1,8 @@
105 # maas - txlongpoll service
106 #
107-# MaaS Provisioning Service txlongpoll
108+# MAAS Provisioning Service txlongpoll
109
110-description "MaaS"
111+description "MAAS txlongpoll"
112 author "Andres Rodriguez <andres.rodriguez@canonical.com>"
113
114 start on filesystem and net-device-up
115
116=== modified file 'debian/maas.postinst'
117--- debian/maas.postinst 2012-03-14 23:29:38 +0000
118+++ debian/maas.postinst 2012-03-15 19:43:24 +0000
119@@ -35,16 +35,16 @@
120 # Give appropriate permissions
121 if [ ! -f /var/log/maas/maas.log ]; then
122 touch /var/log/maas/maas.log
123- chown -R root:www-data /var/log/maas
124- chmod 620 /var/log/maas/maas.log
125- chmod -R 775 /var/log/maas/oops
126 fi
127+ chown -R root:www-data /var/log/maas
128+ chmod 620 /var/log/maas/maas.log
129+ chmod -R 775 /var/log/maas/oops
130
131 # Configure database
132 dbc_go maas $@
133- if grep -qs "^\ \{1,\} 'PASSWORD': '[a-zA-Z0-9]\{0,\}',$" /usr/share/maas/maas_local_settings.py; then
134+ if grep -qs "^\ \{1,\} 'PASSWORD': '[a-zA-Z0-9]\{0,\}',$" /etc/maas/maas_local_settings.py; then
135 sed -i "s/^\ \{1,\} 'PASSWORD': '[a-zA-Z0-9]\{0,\}',$/ 'PASSWORD': '"$dbc_dbpass"',/" \
136- /usr/share/maas/maas_local_settings.py
137+ /etc/maas/maas_local_settings.py
138 fi
139 if $syncdb; then
140 maas syncdb --noinput
141
142=== modified file 'debian/patches/01-fix-database-settings.patch'
143--- debian/patches/01-fix-database-settings.patch 2012-03-14 23:29:38 +0000
144+++ debian/patches/01-fix-database-settings.patch 2012-03-15 19:43:24 +0000
145@@ -1,7 +1,7 @@
146-Index: maas-0.1+bzr281+dfsg.orig/contrib/maas_local_settings_sample.py
147+Index: maas-0.1+bzr291+dfsg.orig/contrib/maas_local_settings_sample.py
148 ===================================================================
149---- maas-0.1+bzr281+dfsg.orig.orig/contrib/maas_local_settings_sample.py 2012-03-14 14:29:12.145287076 -0400
150-+++ maas-0.1+bzr281+dfsg.orig/contrib/maas_local_settings_sample.py 2012-03-14 14:29:16.421288467 -0400
151+--- maas-0.1+bzr291+dfsg.orig.orig/contrib/maas_local_settings_sample.py 2012-03-15 14:07:36.830510908 -0400
152++++ maas-0.1+bzr291+dfsg.orig/contrib/maas_local_settings_sample.py 2012-03-15 14:07:39.286510367 -0400
153 @@ -7,7 +7,7 @@
154 DEFAULT_MAAS_URL = "http://maas.internal.example.com/"
155
156@@ -9,7 +9,7 @@
157 -STATIC_ROOT = '/var/lib/maas/static/'
158 +STATIC_ROOT = '/usr/share/maas/web/static/'
159
160- # Prefix to use for MaaS's urls.
161+ # Prefix to use for MAAS's urls.
162 # If FORCE_SCRIPT_NAME is None (the default), all the urls will start with
163 @@ -56,9 +56,9 @@
164 'default': {
165
166=== removed directory 'debian/po'
167=== removed file 'debian/po/POTFILES.in'
168--- debian/po/POTFILES.in 2012-03-09 22:55:04 +0000
169+++ debian/po/POTFILES.in 1970-01-01 00:00:00 +0000
170@@ -1,1 +0,0 @@
171-[type: gettext/rfc822deb] maas.templates
172
173=== removed file 'debian/po/templates.pot'
174--- debian/po/templates.pot 2012-03-09 22:55:04 +0000
175+++ debian/po/templates.pot 1970-01-01 00:00:00 +0000
176@@ -1,46 +0,0 @@
177-# SOME DESCRIPTIVE TITLE.
178-# Copyright (C) YEAR THE PACKAGE'S COPYRIGHT HOLDER
179-# This file is distributed under the same license as the PACKAGE package.
180-# FIRST AUTHOR <EMAIL@ADDRESS>, YEAR.
181-#
182-#, fuzzy
183-msgid ""
184-msgstr ""
185-"Project-Id-Version: maas\n"
186-"Report-Msgid-Bugs-To: maas@packages.debian.org\n"
187-"POT-Creation-Date: 2012-03-09 12:50-0500\n"
188-"PO-Revision-Date: YEAR-MO-DA HO:MI+ZONE\n"
189-"Last-Translator: FULL NAME <EMAIL@ADDRESS>\n"
190-"Language-Team: LANGUAGE <LL@li.org>\n"
191-"Language: \n"
192-"MIME-Version: 1.0\n"
193-"Content-Type: text/plain; charset=CHARSET\n"
194-"Content-Transfer-Encoding: 8bit\n"
195-
196-#. Type: string
197-#. Description
198-#: ../maas.templates:1001
199-msgid "Create administrator username for MaaS:"
200-msgstr ""
201-
202-#. Type: string
203-#. Description
204-#: ../maas.templates:1001
205-msgid ""
206-"MaaS requires to set an administrative user in order to be able to gain "
207-"access to the MaaS management WebUI."
208-msgstr ""
209-
210-#. Type: password
211-#. Description
212-#: ../maas.templates:2001
213-msgid "Set password for the administrator username:"
214-msgstr ""
215-
216-#. Type: password
217-#. Description
218-#: ../maas.templates:2001
219-msgid ""
220-"It is highly recommendaed that you set a password for the administrative "
221-"user that has just been created."
222-msgstr ""
223
224=== modified file 'debian/rules'
225--- debian/rules 2012-03-14 23:29:38 +0000
226+++ debian/rules 2012-03-15 19:43:24 +0000
227@@ -11,9 +11,9 @@
228 dh_auto_install
229
230 # Copy the local config file
231- install -d -m 755 $(CURDIR)/debian/tmp/usr/share/maas
232+ install -d -m 755 $(CURDIR)/debian/tmp/etc/maas
233 cp $(CURDIR)/contrib/maas_local_settings_sample.py \
234- $(CURDIR)/debian/tmp/usr/share/maas/maas_local_settings.py
235+ $(CURDIR)/debian/tmp/etc/maas/maas_local_settings.py
236
237 # Move static files
238 install -d -m 755 $(CURDIR)/debian/tmp/usr/share/maas/web/static

Subscribers

People subscribed via source and target branches

to all changes: