From owner-freebsd-sparc64@FreeBSD.ORG Fri Mar 25 09:03:28 2005 Return-Path: Delivered-To: freebsd-sparc64@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 81B9C16A4CE for ; Fri, 25 Mar 2005 09:03:28 +0000 (GMT) Received: from citadel.icyb.net.ua (citadel.icyb.net.ua [212.40.38.140]) by mx1.FreeBSD.org (Postfix) with ESMTP id CCC9343D5E for ; Fri, 25 Mar 2005 09:03:26 +0000 (GMT) (envelope-from avg@icyb.net.ua) Received: from [212.40.38.87] (oddity-e.topspin.kiev.ua [212.40.38.87]) by citadel.icyb.net.ua (8.8.8p3/ICyb-2.3exp) with ESMTP id LAA23317 for ; Fri, 25 Mar 2005 11:03:24 +0200 (EET) (envelope-from avg@icyb.net.ua) Message-ID: <4243D3DC.3030508@icyb.net.ua> Date: Fri, 25 Mar 2005 11:03:24 +0200 From: Andriy Gapon User-Agent: Mozilla Thunderbird 1.0 (X11/20050319) X-Accept-Language: en-us, en MIME-Version: 1.0 To: freebsd-sparc64@freebsd.org References: <42415375.4080107@icyb.net.ua> <4241A28C.4060309@icyb.net.ua> <20050324200719.GD60666@dragon.NUXI.org> In-Reply-To: <20050324200719.GD60666@dragon.NUXI.org> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Subject: Re: udfclient-0.3.3_1 failed on sparc64: bswap.h X-BeenThere: freebsd-sparc64@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: Porting FreeBSD to the Sparc List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 25 Mar 2005 09:03:28 -0000 on 24.03.2005 22:07 David O'Brien said the following: > On Wed, Mar 23, 2005 at 07:08:28PM +0200, Andriy Gapon wrote: >>+++ ./work/UDFclient.0.3.3/udf_bswap.h Wed Mar 23 19:00:10 2005 >>@@ -52,8 +52,33 @@ >> /* rest only relevant for big endian machines */ >> #if (BYTE_ORDER == BIG_ENDIAN) >> >>+#if !defined(__FreeBSD__) >> #include >> #include >>+#else >>+#if !defined(bswap16) >>+/* this version of FreeBSD does not provide bswap*() functions */ > > > I don't see why this is necessary -- 4.x does not run on our big endian > platforms, and 5.x does have bswap*. > David, the first part of the patch is needed because we do not have bswap.h header file (bswap*() are in endian.h). The second part was just in case (I have not followed any non-x86 developments). It can be omitted then. Thank you for your knowledge. -- Andriy Gapon