Re: [Xen-devel] [PATCH 2 of 2] Make tools/check scripts cross-friendly

["Christoph Egger" <Christoph.Egger@xxxxxxx>]

check_udev fails due to a typo in the patch.
See inline for other comments.

Christoph


On Saturday 19 January 2008 05:02:31 Aron Griffis wrote:
> # HG changeset patch
> # User Aron Griffis <aron@xxxxxx>
> # Date 1200715193 18000
> # Node ID 735710fbd87c8c3701bb2e33746500793acf14bf
> # Parent  79cf8ffbc6678f201d35724bf083ea9749818597
> Make tools/check scripts cross-friendly
>
> This patch introduces tools/check/funcs.sh with some
> cross-friendly abstractions, then modifies the check_*
> scripts to use them.
>
> Signed-off-by: Aron Griffis <aron@xxxxxx>
>

[snip]

> diff -r 79cf8ffbc667 -r 735710fbd87c tools/check/check_udev
> --- a/tools/check/check_udev  Fri Jan 18 22:59:53 2008 -0500
> +++ b/tools/check/check_udev  Fri Jan 18 22:59:53 2008 -0500
> @@ -1,34 +1,19 @@
>  #!/bin/sh
>  # CHECK-INSTALL
>
> -RC=0
> +. funcs.sh
>
> -case ${OS} in
> +case $OS in
>  OpenBSD|NetBSD|FreeBSD)
> -     TOOL="vnconfig"
> -     which ${TOOL} 1>/dev/null 2>&1 || RC=1
> +     has_or_fail vncconfig
                            ^^^^^^
Typo!!

>       ;;
>  Linux)
> -     TOOL="udevinfo"
> -     UDEV_VERSION="0"
> -     test -x "$(which ${TOOL} 2>/dev/null)" && \
> -             UDEV_VERSION=$(${TOOL} -V | sed -e 's/^[^0-9]*
> \([0-9]\{1,\}\)[^0-9]\{0,\}/\1/') -   if test "${UDEV_VERSION}" -ge 059; then
> -             RC=0
> -     else
> -             TOOL="hotplug"
> -             which ${TOOL} 1>/dev/null 2>&1 || RC=1
> -     fi
> +     has_or_fail udevinfo
> +     [ "`udevinfo -V | awk '{print $NF}'`" -ge 59 ] 2>/dev/null || \
> +             has hotplug || \
> +             fail "udev is too old, upgrade to version 59 or later"
>       ;;
>  *)
> -     TOOL=""
> -     echo "Unknown OS" && RC=1
> +     fail "unknown OS"
>       ;;
>  esac
> -
> -if test ${RC} -ne 0; then
> -     echo
> -     echo " *** Check for ${TOOL} FAILED"
> -fi
> -
> -exit ${RC}
> diff -r 79cf8ffbc667 -r 735710fbd87c tools/check/check_x11_devel
> --- a/tools/check/check_x11_devel     Fri Jan 18 22:59:53 2008 -0500
> +++ b/tools/check/check_x11_devel     Fri Jan 18 22:59:53 2008 -0500
> @@ -1,15 +1,8 @@
>  #!/bin/sh
>  # CHECK-BUILD
>
> -RC=0
> +. funcs.sh
>
> -set -e
> -test -r /usr/include/X11/keysymdef.h || \
> -test -r /usr/X11R6/include/X11/keysymdef.h || RC=1
> -
> -if test ${RC} -ne 0; then
> -     echo
> -     echo " *** Check for x11 headers FAILED"
> -fi
> -
> -exit ${RC}
> +has_header X11/keysymdef.h || \
> +has_header /usr/X11R6/include/X11/keysymdef.h || \
> +fail "can't find X11 headers"
> diff -r 79cf8ffbc667 -r 735710fbd87c tools/check/check_xgettext
> --- a/tools/check/check_xgettext      Fri Jan 18 22:59:53 2008 -0500
> +++ b/tools/check/check_xgettext      Fri Jan 18 22:59:53 2008 -0500
> @@ -1,14 +1,6 @@
>  #!/bin/sh
>  # CHECK-BUILD
>
> -RC=0
> +. funcs.sh
>
> -TOOL="xgettext"
> -which ${TOOL} 1>/dev/null 2>&1 || RC=1
> -
> -if test ${RC} -ne 0; then
> -     echo
> -     echo " *** Check for the gettext utility (${TOOL}) FAILED"
> -fi
> -
> -exit ${RC}
> +has_or_fail xgettext
> diff -r 79cf8ffbc667 -r 735710fbd87c tools/check/check_xml2
> --- a/tools/check/check_xml2  Fri Jan 18 22:59:53 2008 -0500
> +++ b/tools/check/check_xml2  Fri Jan 18 22:59:53 2008 -0500
> @@ -1,5 +1,7 @@
>  #!/bin/sh
>  # CHECK-BUILD CHECK-INSTALL
> +
> +. funcs.sh
>
>  if [ ! "$LIBXENAPI_BINDINGS" = "y" -a ! "$ACM_SECURITY" = "y" ]
>  then
> @@ -7,32 +9,6 @@ then
>      exit 0
>  fi
>
> -RC=0
> -
> -XML2_CONFIG="$(which xml2-config)"
> -tmpfile=$(mktemp)
> -
> -if test -z ${XML2_CONFIG}; then
> -    RC=1
> -else
> -    ${XML2_CONFIG} --libs 2>&1 > /dev/null
> -    RC=$?
> -fi
> -
> -if test $RC -ne 0; then
> -    echo "FAILED"
> -     echo " *** xml2-config is missing. "
> -    echo " *** Please install libxml2-devel."
> -elif ! ld $($XML2_CONFIG --libs) -o $tmpfile >/dev/null 2>&1; then
> -    echo "FAILED"
> -    echo " *** dependency libraries for xml2 are missing: "
> -    RC=1
> -    for i in $(ld $($XML2_CONFIG --libs) -o $tmpfile 2>&1 >/dev/null); do
> -        case $i in
> -        -l*) echo lib${i#-l}
> -        esac
> -    done
> -fi
> -rm -f $tmpfile
> -
> -exit $RC
> +has_or_fail xml2-config
> +xml2_libs=`xml2-config --libs` || fail "xml2-config --libs failed"
> +test_link $xml2_libs || fail "dependency libraries for xml2 are missing"
> diff -r 79cf8ffbc667 -r 735710fbd87c tools/check/check_zlib_devel
> --- a/tools/check/check_zlib_devel    Fri Jan 18 22:59:53 2008 -0500
> +++ b/tools/check/check_zlib_devel    Fri Jan 18 22:59:53 2008 -0500
> @@ -1,14 +1,6 @@
>  #!/bin/sh
>  # CHECK-BUILD
>
> -RC=0
> +. funcs.sh
>
> -set -e
> -test -r /usr/include/zlib.h || RC=1
> -
> -if test ${RC} -ne 0; then
> -     echo
> -     echo " *** Check for zlib headers FAILED"
> -fi
> -
> -exit ${RC}
> +has_header zlib.h || fail "can't find zlib headers"
> diff -r 79cf8ffbc667 -r 735710fbd87c tools/check/check_zlib_lib
> --- a/tools/check/check_zlib_lib      Fri Jan 18 22:59:53 2008 -0500
> +++ b/tools/check/check_zlib_lib      Fri Jan 18 22:59:53 2008 -0500
> @@ -1,24 +1,12 @@
>  #!/bin/sh
>  # CHECK-BUILD CHECK-INSTALL
>
> -RC=0
> +. funcs.sh
>
> -case $(uname -s) in
> +case $OS in
>  FreeBSD|NetBSD|OpenBSD)
>       exit 0
>       ;;
> -*)
> -     ;;
>  esac
>
> -PATH=/sbin:$PATH
> -
> -set -e
> -ldconfig -p 2>&1 | grep -q libz.so || RC=1
> -
> -if test ${RC} -ne 0; then
> -     echo
> -     echo " *** Check for zlib library FAILED"
> -fi
> -
> -exit ${RC}
> +has_lib libz.so || fail "can't find zlib"
> diff -r 79cf8ffbc667 -r 735710fbd87c tools/check/chk
> --- a/tools/check/chk Fri Jan 18 22:59:53 2008 -0500
> +++ b/tools/check/chk Fri Jan 18 22:59:53 2008 -0500
> @@ -7,17 +7,16 @@ func_usage ()
>      echo
>      echo "Check suitability for Xen build or install."
>      echo "Exit with 0 if OK, 1 if not."
> -    echo "Prints only failed tests."
>      echo
>      echo "Calling with 'clean' removes generated files."
>      exit 1
>  }
>
> -PATH=${PATH}:/sbin:/usr/sbin
> +PATH=$PATH:/sbin:/usr/sbin
>  OS=`uname -s`
>  export PATH OS
>
> -if test "${OS}" = "SunOS"; then
> +if [ "$OS" = "SunOS" ]; then

What does this fix? Don't change this if you can't test on Solaris.

>       exit 0
>  fi
>
> @@ -38,7 +37,7 @@ esac
>
>  failed=0
>
> -echo "Xen ${check} " $(date)
> +echo "Xen ${check} " `date`
>  for f in check_* ; do
>      case $f in
>          *~)
> @@ -50,7 +49,7 @@ for f in check_* ; do
>      if ! [ -x $f ] ; then
>          continue
>      fi
> -    if ! grep -q ${check} $f ; then
> +    if ! grep -Fq "$check" $f ; then
>          continue
>      fi
>      echo -n "Checking $f: "
> diff -r 79cf8ffbc667 -r 735710fbd87c tools/check/funcs.sh
> --- /dev/null Thu Jan 01 00:00:00 1970 +0000
> +++ b/tools/check/funcs.sh    Fri Jan 18 22:59:53 2008 -0500
> @@ -0,0 +1,85 @@
> +# has is the same as which, except it handles cross environments
> +has() {
> +     if [ -z "$CROSS_COMPILE" ]; then
> +             command which "$@"
> +             return $?
> +     fi
> +
> +     check_sys_root || return 1
> +
> +     # subshell to prevent pollution of caller's IFS
> +     (
> +     IFS=:
> +     for p in $PATH; do
> +             if [ -x "$CROSS_SYS_ROOT/$p/$1" ]; then
> +                     echo "$CROSS_SYS_ROOT/$p/$1"
> +                     return 0
> +             fi
> +     done
> +     return 1
> +     )
> +}
> +
> +has_or_fail() {
> +     has "$1" >/dev/null || fail "can't find $1"
> +}
> +
> +has_header() {
> +     case $1 in
> +             /*) ;;
> +             *) set -- "/usr/include/$1" ;;
> +     esac
> +
> +     check_sys_root || return 1
> +
> +     test -r "$CROSS_SYS_ROOT$1"
> +     return $?
> +}
> +
> +has_lib() {
> +     check_sys_root || return 1
> +
> +     # subshell to prevent pollution of caller's environment
> +     (
> +     PATH=/sbin:$PATH        # for ldconfig
> +
> +     # This relatively common in a sys-root; libs are installed but
> +     # ldconfig hasn't run there, so ldconfig -p won't work.  We can
> +     # only fix it if it's writeable.
> +     [ "$OS" != Linux ] || \
> +     [ -f "$CROSS_SYS_ROOT/etc/ld.so.cache" ] || \
> +     ldconfig -r "$CROSS_SYS_ROOT" && \
> +     ldconfig -p ${CROSS_SYS_ROOT+-r "$CROSS_SYS_ROOT"} | grep -Fq "$1"
> +     return $?
> +     )
> +}

This can be more readable with "if test" constructs.

> +
> +test_link() {
> +     # subshell to trap removal of tmpfile
> +     (
> +     unset tmpfile
> +     trap 'rm -f "$tmpfile"; exit' 0 1 2 15
> +     tmpfile=`mktemp` || return 1
> +     ld "$@" -o "$tmpfile" >/dev/null 2>&1
> +     return $?
> +     )
> +}
> +
> +# this function is used commonly above
> +check_sys_root() {
> +     [ -z "$CROSS_COMPILE" ] && return 0
> +     if [ -z "$CROSS_SYS_ROOT" ]; then
> +             echo "please set CROSS_SYS_ROOT in the environment"
> +             return 1
> +     fi
> +     if [ ! -d "$CROSS_SYS_ROOT" ]; then
> +             echo "no sys-root found at $CROSS_SYS_ROOT"
> +             return 1
> +     fi
> +}
> +
> +fail() {
> +     echo
> +     echo " *** `basename "$0"` FAILED${*+: $*}"
> +     exit 1
> +}
>
>
>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@xxxxxxxxxxxxxxxxxxx
> http://lists.xensource.com/xen-devel



-- 
AMD Saxony, Dresden, Germany
Operating System Research Center

Legal Information:
AMD Saxony Limited Liability Company & Co. KG
Sitz (Geschäftsanschrift):
   Wilschdorfer Landstr. 101, 01109 Dresden, Deutschland
Registergericht Dresden: HRA 4896
vertretungsberechtigter Komplementär:
   AMD Saxony LLC (Sitz Wilmington, Delaware, USA)
Geschäftsführer der AMD Saxony LLC:
   Dr. Hans-R. Deppe, Thomas McCoy



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