OpenStack Compute (nova)

Merge lp:~gandelman-a/nova/ubuntu-rm-postinst into lp:~openstack-ubuntu-packagers/nova/ubuntu

Proposed by Adam Gandelman on 2011-12-22

Status:	Needs review
Proposed branch:	lp:~gandelman-a/nova/ubuntu-rm-postinst
Merge into:	lp:~openstack-ubuntu-packagers/nova/ubuntu
Diff against target:	31 lines (+4/-12) 2 files modified debian/changelog (+4/-0) debian/python-nova.postinst (+0/-12)
To merge this branch:	bzr merge lp:~gandelman-a/nova/ubuntu-rm-postinst
Related bugs:	Link a bug report

Reviewer	Date Requested	Status
Jay Pipes (community)		Approve on 2012-01-02
OpenStack Ubuntu packagers	2011-12-22	Pending
Review via email: mp+86760@code.launchpad.net

Revision history for this message

Jay Pipes (jaypipes) wrote on 2012-01-02:

lgtm, thx Adam!

review: Approve

205. By Adam Gandelman on 2011-12-22: Removed python-nova.postinst, let dh_python2 generate instead since
python-support is not a dependency. (LP: #907543)

[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:

 === modified file 'debian/changelog'
 --- debian/changelog	2011-12-22 15:16:36 +0000
 +++ debian/changelog	2011-12-22 22:39:23 +0000
@@ -6,6 +6,10 @@
    [ Dan Prince ]
    * debian/nova_sudoers: Fix typo
++  [ Adam Gandelman ]
++  * Removed python-nova.postinst, let dh_python2 generate instead since
++    python-support is not a dependency. (LP: #907543)
++
   -- Thierry Carrez (ttx) <thierry@openstack.org>  Thu, 22 Dec 2011 16:15:22 +0100
  nova (2012.1~e2-0ubuntu1) precise; urgency=low
 === removed file 'debian/python-nova.postinst'
 --- debian/python-nova.postinst	2010-11-22 22:43:46 +0000
 +++ debian/python-nova.postinst	1970-01-01 00:00:00 +0000
@@ -1,12 +0,0 @@
--#!/bin/sh
--
--set -e
--
--#DEBHELPER#
--
--if [ "$1" = "configure" ]
--then
--    update-python-modules --post-install
--fi
--
--exit 0