Re: [Fwd: Re: [pgsql-hackers-win32] Import from Linux to

From: Andrew Dunstan <andrew(at)dunslane(dot)net>
To: PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>, "Patches (PostgreSQL)" <pgsql-patches(at)postgresql(dot)org>
Subject: Re: [Fwd: Re: [pgsql-hackers-win32] Import from Linux to
Date: 2004-08-14 00:39:25
Message-ID: 411D5F3D.4090606@dunslane.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches

Tom Lane wrote:

>Andrew Dunstan <andrew(at)dunslane(dot)net> writes:
>
>
>>>The attached patch appears to solve the problem. However, while it
>>>makes us conform to the first sentence below from the docs, it doesn't
>>>comply with the second. Not sure what to do about that. Maybe there's
>>>a better solution?
>>>
>>>
>
>
>
>>Attached patch seems much better, I think.
>>
>>
>
>I think it is still not quite there. Since as you noted the backend
>will complain if line endings don't match, if we hit EOF then we have
>to cons up a \. line with the correct ending. (BTW, this is not
>actually necessary when talking 3.0 protocol, but it is when talking
>to an older server.)
>
>I modified the patch a little more and applied the attached. It seems
>to work for me but could use more testing.
>
>
>
>

WorksForMe, and looks good. You're right, I had forgotten the EOF case.

Should it be backported for the upcoming stable release(s)? Bruce and I
were discussing this earlier.

Pro: it's an ugly bug and hard to diagnose - things just seem to die for
no apparent reason.
Con: there's a workaround - just make sure to run dos2unix on your file
if necessary.

cheers

andrew

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Simon@2ndquadrant.com 2004-08-14 00:44:57 Re: Development Schedule Page
Previous Message Tom Lane 2004-08-13 23:37:49 Re: Development Schedule Page

Browse pgsql-patches by date

  From Date Subject
Next Message Andreas Seltenreich 2004-08-14 00:46:35 Minor psql reference fix
Previous Message Tom Lane 2004-08-13 23:02:33 Re: [Fwd: Re: [pgsql-hackers-win32] Import from Linux to Windows]