Mailing-List: contact cygwin-help AT cygwin DOT com; run by ezmlm
List-Subscribe: <mailto:cygwin-subscribe AT cygwin DOT com>
List-Archive: <http://sourceware.org/ml/cygwin/>
List-Post: <mailto:cygwin AT cygwin DOT com>
List-Help: <mailto:cygwin-help AT cygwin DOT com>, <http://sourceware.org/ml/#faqs>
Sender: cygwin-owner AT cygwin DOT com
Mail-Followup-To: cygwin AT cygwin DOT com
Delivered-To: mailing list cygwin AT cygwin DOT com
Content-Class: urn:content-classes:message
MIME-Version: 1.0
Content-Type: text/plain; 	charset="iso-8859-1"
Subject: RE: patch to fix broken xdr headers
Date: Mon, 15 Aug 2005 13:32:51 -0400
Message-ID: <3D848382FB72E249812901444C6BDB1D03E0505C@exchange.timesys.com>
From: "Robb, Sam" <sam DOT robb AT timesys DOT com>
To: <cygwin AT cygwin DOT com>
X-IsSubscribed: yes
Content-Transfer-Encoding: 8bit
X-MIME-Autoconverted: from quoted-printable to 8bit by delorie.com id j7FHXOXI009254

> This patch fixes the broken CYGWIN xdr headers under g++.

John,

  Thanks for the effort you put into this - unfortunately, I'm
a bit leery of applying such a significant patch without a
better understanding of what the exact problem is.

  Additionally, it looks like there are a lot of formatting
changes in the patch you provided, ex:

> -	XDR_ENCODE=0,
> 
> -	XDR_DECODE=1,
> 
> -	XDR_FREE=2
> 
> +  XDR_ENCODE = 0,
> 
> +  XDR_DECODE = 1,
> 
> +  XDR_FREE = 2

  This makes me suspect that this is not really a patch, but
just a diff against an existing linux rpc/xdr.h file.  I'd be
happy to update the sunrpc package to match the linux headers,
but I suspect that I would have to update more than just one
file to keep everything consistent. Were you able to rebuild
the sunrpc package from source with these changes in place?

-Samrobb
 

--
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple
Problem reports:       http://cygwin.com/problems.html
Documentation:         http://cygwin.com/docs.html
FAQ:                   http://cygwin.com/faq/