From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Jan Beulich" Subject: Re: [PATCH] yet another package check (take 2) Date: Thu, 14 Dec 2006 16:59:36 +0000 Message-ID: <45819108.76E4.0078.0@novell.com> References: <45818894.76E4.0078.0@novell.com> <20061214164716.GA2717@leeni.uk.xensource.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20061214164716.GA2717@leeni.uk.xensource.com> Content-Disposition: inline List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: Ewan Mellor Cc: Christoph Egger , xen-devel@lists.xensource.com List-Id: xen-devel@lists.xenproject.org >How about > >if ! python -c 'import xml.dom.minidom' 2>/dev/null ; then > echo > echo " *** Check for python development environment FAILED" > exit 1 >else > exit 0 >fi Why not - I didn't invent the other method, I just cloned what we had for python-devel (which in turn I created based on suggestions from you and others). I'm not a python person at all, I just want understandable error messages when dependencies are missing instead of cryptic ones. Jan