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
MIME-Version:	1.0
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
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/

- Raw text -