This is the mail archive of the
cygwin-apps
mailing list for the Cygwin project.
Re: [PATCH libtirpc] Disable libtirpc's own bindresvport{,_sa}() in favor of Cygwin's
- From: Brian Inglis <Brian dot Inglis at SystematicSw dot ab dot ca>
- To: cygwin-apps at cygwin dot com
- Date: Wed, 7 Feb 2018 11:51:29 -0700
- Subject: Re: [PATCH libtirpc] Disable libtirpc's own bindresvport{,_sa}() in favor of Cygwin's
- Authentication-results: sourceware.org; auth=none
- References: <59D90AF8D70E9740907BACDE2BCB520836E4AA63@RESW102.resdom01.local> <20180206142012.GB30794@calimero.vinschen.de> <80645451-e2d6-56bf-87c4-c02ac801f201@maxrnd.com> <af20a08b-f826-d2ff-fe93-cf6a72677aa5@maxrnd.com> <29e2c298-107a-eb7e-31a9-0b937a7d95db@redhat.com>
- Reply-to: Brian dot Inglis at SystematicSw dot ab dot ca
On 2018-02-07 08:38, Eric Blake wrote:
> On 02/07/2018 01:29 AM, Mark Geisert wrote:
>> I don't have libtirpc in git so I'm submitting a text patch. Sorry for any
>> inconvenience. This is Cygwin-specific and against src/bindresvport.c of
>> libtirpc 1.0.1. Unsure if it ought to go upstream; appreciate input on that.
>> Thanks much,
>>
>> ..mark
>>
>> --------8<--------
>> 35a36,38
>> > /* On Cygwin prefer Cygwin's bindresvport{,_sa}() to portable version here */
>
> An ed-script diff is practically useless; without context, it is too easy to
> misapply the patch if the file has been edited differently in the meantime.
> ALWAYS use 'diff -u' (what git does by default) or 'diff -c' when generating a
> patch, so that it has proper context.
Also mandatory to add -p, --show-c-function for patches, and in general for
directory or recursive patch diffs -N, --new-file so new files are diffed as if
against an empty file; --strip-trailing-cr is useful if some files may have CRs.
--
Take care. Thanks, Brian Inglis, Calgary, Alberta, Canada