Tom Lane wrote:
Andrew Dunstan [EMAIL PROTECTED] writes:
No, I think 7.4 should do. 7.3 users will still have the dos2unix workaround
available. Are you going to do the 7.4 patch, or do you need me to? I
normally only keep a HEAD tree checked out. A quick look at the cvsweb diffs
suggests the
Andrew Dunstan wrote:
Tom Lane wrote:
Andrew Dunstan [EMAIL PROTECTED] writes:
No, I think 7.4 should do. 7.3 users will still have the dos2unix workaround
available. Are you going to do the 7.4 patch, or do you need me to? I
normally only keep a HEAD tree checked out. A quick
Bruce Momjian wrote:
One issue is that pre-8.0, psql files were opened in Win32 text mode, so
we wouldn't have seen this bug on Win32, but we would on Linux.
Because we open them on Win32 now in binary mode so we see control-Z it
will show up on Win32 too.
true, *BUT*
The patch is not
Andrew Dunstan wrote:
Bruce Momjian wrote:
One issue is that pre-8.0, psql files were opened in Win32 text mode, so
we wouldn't have seen this bug on Win32, but we would on Linux.
Because we open them on Win32 now in binary mode so we see control-Z it
will show up on Win32 too.
Andrew Dunstan [EMAIL PROTECTED] writes:
Here's the diff with 7.4 line numbers - tests fine for me.
Patch applied. Thanks for double-checking it.
regards, tom lane
---(end of broadcast)---
TIP 3: if posting/reading
Andrew Dunstan [EMAIL PROTECTED] writes:
The patch is not platform-specific. It simply makes psql accept the same
line endings on COPY FROM that the backend will accept - in effect it
makes it line-end agnostic - this is a Good Thing (tm).
Strictly speaking it's not there yet --- psql still
Andrew Dunstan [EMAIL PROTECTED] 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
Tom Lane wrote:
Andrew Dunstan [EMAIL PROTECTED] 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?
Andrew Dunstan [EMAIL PROTECTED] writes:
Should it be backported for the upcoming stable release(s)? Bruce and I
were discussing this earlier.
Probably a good idea, since we do support psql on Windows even in the
older releases.
My personal opinion is to back-port only as far as 7.4, but if
Tom Lane said:
Andrew Dunstan [EMAIL PROTECTED] writes:
Should it be backported for the upcoming stable release(s)? Bruce and
I were discussing this earlier.
Probably a good idea, since we do support psql on Windows even in the
older releases.
My personal opinion is to back-port only as
Andrew Dunstan [EMAIL PROTECTED] writes:
No, I think 7.4 should do. 7.3 users will still have the dos2unix workaround
available. Are you going to do the 7.4 patch, or do you need me to? I
normally only keep a HEAD tree checked out. A quick look at the cvsweb diffs
suggests the patch should
11 matches
Mail list logo
