Re: [Xen-devel] [PATCH] yet another package check (take 2)

[Ewan Mellor <ewan@xxxxxxxxxxxxx>]

On Thu, Dec 14, 2006 at 04:23:32PM +0000, Jan Beulich wrote:

> As before, the file added to tools/check/ must be given exec permission.
> 
> Now using the approach suggested by Christoph Egger <Christoph.Egger@xxxxxxx>.
>
> [Snip]
>
> Index: 2006-12-11/tools/check/check_python_xml
> ===================================================================
> --- /dev/null 1970-01-01 00:00:00.000000000 +0000
> +++ 2006-12-11/tools/check/check_python_xml   2006-12-14 17:04:04.000000000 
> +0100
> @@ -0,0 +1,19 @@
> +#!/bin/sh
> +# CHECK-INSTALL
> +
> +RC=0
> +
> +python -c '
> +import os.path, sys
> +for p in sys.path:
> +     if os.path.exists(p + "/xml/dom/minidom.py"):
> +             sys.exit(0)
> +sys.exit(1)
> +' || RC=1
> +
> +if test ${RC} -ne 0; then
> +     echo
> +     echo " *** Check for python development environment FAILED"
> +fi
> +
> +exit ${RC}

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

Ewan.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel