launchpad-reviewers team mailing list archive

[Julian Edwards <julian.edwards@xxxxxxxxxxxxx>]

Julian Edwards has proposed merging lp:~julian-edwards/maas/packaging into lp:~maas-maintainers/maas/packaging.

Commit message:
Fix whacko shell  crap yet again.

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)
Related bugs:
  Bug #1052805 in MAAS: "Packaging build broken: no file to patch"
  https://bugs.launchpad.net/maas/+bug/1052805

For more details, see:
https://code.launchpad.net/~julian-edwards/maas/packaging/+merge/164825
-- 
https://code.launchpad.net/~julian-edwards/maas/packaging/+merge/164825
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~julian-edwards/maas/packaging into lp:~maas-maintainers/maas/packaging.

=== modified file 'debian/maas-dhcp.maas-dhcp-server.upstart'
--- debian/maas-dhcp.maas-dhcp-server.upstart	2013-05-20 03:18:49 +0000
+++ debian/maas-dhcp.maas-dhcp-server.upstart	2013-05-21 02:15:35 +0000
@@ -48,7 +48,7 @@
     # the daemon will run under an unprivileged user).
     # In Precise, ownership was supposed to be dhcp:dhcp.
     mkdir -p $LEASES_DIR
-    if "$RELEASE" == "precise"; then
+    if [ "$RELEASE" == "precise" ]; then
         owner_group="dhcpd:dhcpd"
         dhcpd_owner_opts="-user dhcpd -group dhcpd"
     else