From d97b64b6e69e7ac428e578ec148d93e4687a7ffe Mon Sep 17 00:00:00 2001 From: "Daniel P. Berrange" Date: Thu, 5 Jan 2012 13:15:15 +0000 Subject: [PATCH] Tell users what is about to be installed via sudo Rather than just giving users the sudo password prompt immediately, actually tell them what is about to be installed, so they know whether it is reasonable to be entering their password. Change-Id: Ic0c1de812be119384753895531a008075b13494e Signed-off-by: Daniel P. Berrange --- Authors | 1 + tools/install_venv.py | 2 ++ 2 files changed, 3 insertions(+) diff --git a/Authors b/Authors index d8cf9d7a0a8c..e6eda9240387 100644 --- a/Authors +++ b/Authors @@ -34,6 +34,7 @@ Cor Cornelisse Cory Wright Dan Prince Dan Wendlandt +Daniel P. Berrange Dave Walker David Pravec David Subiros diff --git a/tools/install_venv.py b/tools/install_venv.py index 6dfcb3f17b64..0a9fe8252f98 100644 --- a/tools/install_venv.py +++ b/tools/install_venv.py @@ -106,6 +106,7 @@ class UbuntuOneiric(Distro): """ The pip installed version of m2crypto has problems on oneiric """ + print "Attempting to install 'python-m2crypto' via apt-get" run_command(['sudo', 'apt-get', 'install', '-y', "python-m2crypto"]) @@ -120,6 +121,7 @@ class Fedora(Distro): check_exit_code=False)[1] == 0 def yum_install(self, pkg, **kwargs): + print "Attempting to install '%s' via yum" % pkg run_command(['sudo', 'yum', 'install', '-y', pkg], **kwargs) def apply_patch(self, originalfile, patchfile):