Re: log files and permissions

I checked "log_file_mode GUC" patch, and found a couple of Windows-specific
and translation issues.

* fchmod() is not available on some platforms, including Windows.   fh = fopen(filename, mode);   setvbuf(fh, NULL,
LBF_MODE,0);   fchmod(fileno(fh), Log_file_mode);
 

I think umask()->fopen() is better rather than fopen()->chmod().
See codes in DoCopyTo() at commands/copy.c.

* How does the file mode work on Windows?
If it doesn't work, we should explain it in docs.
Description for .pgpass for Windows might be a help.
| http://developer.postgresql.org/pgdocs/postgres/libpq-pgpass.html
| On Microsoft Windows, ... no special permissions check is made.

* This message format is hard to translate.   ereport(am_rotating ? LOG : FATAL,         (errcode_for_file_access(),
     (errmsg("could not create%slog file \"%s\": %m",                  am_rotating ? " new " : " ", filename))));
 

It might look a duplication of codes, but I think this form is better
because we can reuse the existing translation catalogs.   if (am_rotating)       ereport(FATAL, ... "could not create
logfile ...);   else       ereport(LOG, ... "could not open new log file ...);
 

-- 
Itagaki Takahiro