[PATCH] xen-tools dependency check fix

[PATCH] xen-tools dependency check fix

[Christoph Egger]

[-- Attachment #1: Type: text/plain, Size: 594 bytes --]


Hi!

zlib and crypto libs are part of the base system in *BSD.
So no need to check for them on *BSD.

Signed-off-by: Christoph Egger <Christoph.Egger@amd.com>


-- 
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

[-- Attachment #2: tools_check.diff --]
[-- Type: text/plain, Size: 670 bytes --]

diff -r ad339d88639d tools/check/check_crypto_lib
--- a/tools/check/check_crypto_lib	Mon Sep 24 21:52:10 2007 +0100
+++ b/tools/check/check_crypto_lib	Tue Sep 25 10:50:26 2007 +0000
@@ -2,6 +2,14 @@
 # CHECK-BUILD CHECK-INSTALL
 
 RC=0
+
+case $(uname -s) in
+FreeBSD|NetBSD|OpenBSD)
+	exit 0
+	;;
+*)
+	;;
+esac
 
 PATH=/sbin:$PATH
 set -e
diff -r ad339d88639d tools/check/check_zlib_lib
--- a/tools/check/check_zlib_lib	Mon Sep 24 21:52:10 2007 +0100
+++ b/tools/check/check_zlib_lib	Tue Sep 25 10:50:26 2007 +0000
@@ -2,6 +2,14 @@
 # CHECK-BUILD CHECK-INSTALL
 
 RC=0
+
+case $(uname -s) in
+FreeBSD|NetBSD|OpenBSD)
+	exit 0
+	;;
+*)
+	;;
+esac
 
 PATH=/sbin:$PATH
 

[-- Attachment #3: Type: text/plain, Size: 138 bytes --]

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

Re: [PATCH] xen-tools dependency check fix

[Keir Fraser]

On 25/9/07 09:50, "Christoph Egger" <Christoph.Egger@amd.com> wrote:

> zlib and crypto libs are part of the base system in *BSD.
> So no need to check for them on *BSD.

The checks should succeed though, right? So why not just leave the scripts
as they are?

 -- Keir

Re: [PATCH] xen-tools dependency check fix

[Christoph Egger]

On Tuesday 25 September 2007 10:57:19 Keir Fraser wrote:
> On 25/9/07 09:50, "Christoph Egger" <Christoph.Egger@amd.com> wrote:
> > zlib and crypto libs are part of the base system in *BSD.
> > So no need to check for them on *BSD.
>
> The checks should succeed though, right? So why not just leave the scripts
> as they are?

Basically yes, but the non-portable command is ldconfig. Different options, 
different outputs, even between the different BSDs.
So the patch goes the easiest way to deal with that.



-- 
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

Re: [PATCH] xen-tools dependency check fix

[Keir Fraser]

On 25/9/07 10:17, "Christoph Egger" <Christoph.Egger@amd.com> wrote:

>> The checks should succeed though, right? So why not just leave the scripts
>> as they are?
> 
> Basically yes, but the non-portable command is ldconfig. Different options,
> different outputs, even between the different BSDs.
> So the patch goes the easiest way to deal with that.

Ah, ok.

 K.