Re: [PATCH] Two VMS configuration nits

Frediano Ziglio <[email protected]>
Newsgroups gmane.comp.db.tds.freetds
Message-ID <CAHt6W4dkg77U-6NUzGs46+GeEMcjE-8N58FvuYdYCSoCBq=d6g@mail.gmail.com>
Applied

Frediano


2015-06-14 20:23 GMT+01:00 Craig A. Berry <[email protected]>:
>
>> On Jun 14, 2015, at 11:53 AM, Frediano Ziglio <[email protected]> wrote:
>>
>> Il 12/Giu/2015 19:48, "Craig A. Berry" <[email protected]> ha scritto:
>>>
>>> There are two attached patches.  One avoids a redefinition of a symbol in
>> the VMS-specific Makefile equivalent which bothers one VMS make utility
>> (MMS) but not another (MMK).  The other enables the replacement socketpair
>> for older VMS systems that don’t have their own.  These are against master
>> but should be quite safe for the 0.95 branch as well.
>>>
>>> Thanks to Norm Lastovica for problem reports and testing.
>>>
>>
>> Looks good. Did you try to force socket pair replacement to be used. In
>> Linux it worked but to remove a warning I had to add unistd.h include. Do
>> you want me to add to 0.95 too? Should you/me add a comment in changeset
>> comment about Norm?
>>
>
> I believe there was a warning about close() not being declared, so yes, unistd.h should take care of that.  But it did work despite the warning.  And yes, they should be applied to 0.95 as well as master.  Good call on crediting Norm.  I’ve attached a revised version of patch #0002 with that added to the commit message.
>
> ________________________________________
> Craig A. Berry
> mailto:[email protected]
>
> "... getting out of a sonnet is much more
>  difficult than getting in."
>                  Brad Leithauser
>
> _______________________________________________
> FreeTDS mailing list
> [email protected]
> http://lists.ibiblio.org/mailman/listinfo/freetds
>
_______________________________________________
FreeTDS mailing list
[email protected]
http://lists.ibiblio.org/mailman/listinfo/freetds
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.