Control: -1 +patch
Thomas Goirand <z...@debian.org> writes: > On 09/17/2015 06:36 PM, Gaudenz Steinlin wrote: >> Michael Fincham <michael.finc...@catalyst.net.nz> writes: >> >>> Package: openstack-debian-images >>> Version: 1.4 >>> Severity: normal >>> >>> Dear maintainer, >>> >>> Previously the phrase "Aptitude is the current preferred package management >>> tool for the Debian system." appeared in the Debian documentation. >>> >>> Is the omission of aptitude from the images built by >>> openstack-debian-image an accident? If so, can it be added? >> >> "Unfortunately" Thomas already went ahead and added aptitude to the >> image. You don't say which documentation you are refering to. Is this >> current documentation for jessie from Debian? If so it should be fixed. >> >> Aptitude is not part of the Debian default install anymore and thus >> should not be part of the OpenStack images either. Thomas would you >> consider removing it again? > > Yes, I would. But I have no time to work on this, and I also consider > this a very minor issue (and frankly, I don't care that much about it). > So I very much would welcome anyone to do the work and fix. Do you want > to do it, Gaudenz? Patch attached. If you give me permission, I can also push this directly to the repo. Gaudenz
From 2d3413e043dd3c8bc7a45470923d41fe550d2474 Mon Sep 17 00:00:00 2001
From: Gaudenz Steinlin <gaud...@soziologie.ch>
Date: Fri, 18 Sep 2015 16:33:25 +0200
Subject: [PATCH] Remove aptitude from package list
It's not installed by debootstrap by default and the cloud images should
be as similar to a normal Debian install as possible. There is no need
for two package managers in cloud images.
---
build-openstack-debian-image | 2 +-
debian/changelog | 7 +++++++
2 files changed, 8 insertions(+), 1 deletion(-)
diff --git a/build-openstack-debian-image b/build-openstack-debian-image
index b850e04..7d2fa36 100755
--- a/build-openstack-debian-image
+++ b/build-openstack-debian-image
@@ -158,7 +158,7 @@ if [ -z "${USER_LOGIN}" ] ; then
USER_LOGIN=debian
fi
-NEEDED_PACKAGES=sudo,adduser,locales,extlinux,openssh-server,linux-image-amd64,euca2ools,file,kbd,aptitude
+NEEDED_PACKAGES=sudo,adduser,locales,extlinux,openssh-server,linux-image-amd64,euca2ools,file,kbd
if [ "${RELEASE}" = "wheezy" ] ; then
# These are needed by cloud-init and friends, and since we don't want backports of them,
# but just normal packages from Wheezy, we resolve dependencies by hand, prior to using
diff --git a/debian/changelog b/debian/changelog
index b96faaf..716bd46 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,10 @@
+openstack-debian-images (1.7) UNRELEASED; urgency=medium
+
+ * Remove aptitude from package list. It's not installed by default by
+ debootstrap.
+
+ -- Gaudenz Steinlin <gaud...@debian.org> Fri, 18 Sep 2015 16:32:11 +0200
+
openstack-debian-images (1.6) unstable; urgency=medium
* Added aptitude as installed packages. (Closes: #797962)
--
2.5.1
signature.asc
Description: PGP signature
