Re: [pgsql-hackers-win32] win32 build and test issues - Mailing list pgsql-patches

From Bruce Momjian
Subject Re: [pgsql-hackers-win32] win32 build and test issues
Date
Msg-id 200405020426.i424Qla14068@candle.pha.pa.us
Whole thread Raw
List pgsql-patches
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@postgresql.org
>

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

pgsql-patches by date:

Previous
From: Bruce Momjian
Date:
Subject: Re: FW: Timezone library
Next
From: Bruce Momjian
Date:
Subject: Re: mingw regression test fixes