Re: win32 build and test issues

Andrew Dunstan wrote:
>
>
> Bruce,
>
> the setvbuf patch for startup.c should be applied, as all it does is
> ensure well understood and expected (i.e. = Unix) behaviour for stderr on
> Win32.

OK, this first part of the patch was applied. I will discard the second
part and wait for your continued research. Thanks.

---------------------------------------------------------------------------

>
> I am not happy about the patch for print.c unless I can work out *why* it
> works, or someone can explain it to me. (That's why i made the comment
> below about not applying it, and didn't send it to -patches.)
>
> cheers
>
> andrew
>
>
>
> Bruce Momjian said:
> >
> > Your patch has been added to the PostgreSQL unapplied patches list at:
> >
> > http://momjian.postgresql.org/cgi-bin/pgpatches
> >
> > I will try to apply it within the next 48 hours.
> >
> > -------------------------------------------------------------------------
> --
> >
> >
> > Andrew Dunstan wrote:
> >> Magnus Hagander wrote:
> >>
> >> >>I'm also thinking of creating a simple commandline interface
> >> >>for pqkill
> >> >>that can be used to make a clean shutdown.
> >> >>
> >> >>
> >> >
> >> >See the win32 status page, there is a link to one already.
> >> >
> >> >//Magnus
> >> >
> >> >
> >> >
> >>
> >> I will check it out. Meanwhile, there's promising news. With the patch
> >> below applied to psql, I can run "make check" under MinGW and get all
> >> but 12 tests passing. The remainder appear to be all related to the
> >> timestamp issue, the floating point scientific format issue, and the
> >> join sort order issue, which have been previously noted.
> >>
> >> I'm not suggesting applying the patch yet - I'd like to know why the
> >> line end thing cares what platform it is using - presumably regardless
> >> of where it runs it is still using MSVCRT.
> >>
> >> cheers
> >>
> >> andrew
> >>
> >>
> >>
> >> Index: startup.c
> >> ===================================================================
> >> RCS file: /projects/cvsroot/pgsql-server/src/bin/psql/startup.c,v
> >> retrieving revision 1.91
> >> diff -c -w -r1.91 startup.c
> >> *** startup.c 22 Apr 2004 14:34:38 -0000 1.91
> >> --- startup.c 30 Apr 2004 16:44:45 -0000
> >> ***************
> >> *** 124,129 ****
> >> --- 124,132 ----
> >> }
> >> }
> >>
> >> + #ifdef WIN32
> >> + setvbuf(stderr,NULL,_IONBF,0);
> >> + #endif
> >> pset.cur_cmd_source = stdin;
> >> pset.cur_cmd_interactive = false;
> >> pset.encoding = PQenv2encoding();
> >> Index: print.c
> >> ===================================================================
> >> RCS file: /projects/cvsroot/pgsql-server/src/bin/psql/print.c,v
> >> retrieving revision 1.46
> >> diff -c -w -r1.46 print.c
> >> *** print.c 24 Jan 2004 20:43:26 -0000 1.46
> >> --- print.c 30 Apr 2004 16:44:46 -0000
> >> ***************
> >> *** 388,394 ****
> >> --- 388,396 ----
> >> for (ptr = footers; *ptr; ptr++)
> >> fprintf(fout, "%s\n", *ptr);
> >>
> >> + #ifndef WIN32
> >> fputc('\n', fout);
> >> + #endif
> >>
> >> /* clean up */
> >> free(cell_w);
> >>
>
>
>
>
> ---------------------------(end of broadcast)---------------------------
> TIP 1: subscribe and unsubscribe commands go to majordomo(at)postgresql(dot)org
>

--
Bruce Momjian | http://candle.pha.pa.us
pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073