database/postgres
database/postgres
1
0
Fork 0
mirror of https://github.com/postgres/postgres.git synced 2025-05-18 17:41:14 +03:00
Code

In initdb, ensure stdout/stderr buffering behavior is what we expect.

Browse Source 
Since this program may print to either stdout or stderr, the relative
ordering of its messages depends on the buffering behavior of those files.
Force stdout to be line-buffered and stderr to be unbuffered, ensuring
that the behavior will match standard Unix interactive behavior, even
when stdout and stderr are rerouted to a file.

Per complaint from Tomas Vondra.  The particular case he pointed out is
new in HEAD, but issues of the same sort could arise in any branch with
other error messages, so back-patch to all branches.

I'm unsure whether we might not want to do this in other client programs
as well.  For the moment, just fix initdb.
This commit is contained in:
Tom Lane 2014-05-14 21:13:56 -04:00
parent e7a9020939
commit 4511e6c831
1 changed files with 9 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
src/bin/initdb/initdb.c
View File

@ -3447,6 +3447,15 @@ main(int argc, char *argv[])
char *effective_user; char *effective_user;
char bin_dir[MAXPGPATH]; char bin_dir[MAXPGPATH];
/*
* Ensure that buffering behavior of stdout and stderr matches what it is
* in interactive usage (at least on most platforms). This prevents
* unexpected output ordering when, eg, output is redirected to a file.
* POSIX says we must do this before any other usage of these files.
*/
setvbuf(stdout, NULL, _IOLBF, 0);
setvbuf(stderr, NULL, _IONBF, 0);
progname = get_progname(argv[0]); progname = get_progname(argv[0]);
set_pglocale_pgservice(argv[0], PG_TEXTDOMAIN("initdb")); set_pglocale_pgservice(argv[0], PG_TEXTDOMAIN("initdb"));