Re: spurious use of %m format in pg_upgrade

From: Bruce Momjian <bruce(at)momjian(dot)us>
To: Peter Eisentraut <peter_e(at)gmx(dot)net>
Cc: pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: spurious use of %m format in pg_upgrade
Date: 2011-07-12 15:07:00
Message-ID: 201107121507.p6CF70113152@momjian.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Peter Eisentraut wrote:
> pg_upgrade's pg_scandir_internal() makes use of the non-standard %m
> format:
>
> pg_log(PG_FATAL, "could not open directory \"%s\": %m\n", dirname);
>
> Is this an oversight, or is there an undocumented assumption that this
> code will only be used on platforms where %m works?
>
> (Which platforms don't have scandir() anyway?)

Yes, surely an oversight, and I see it has been removed --- good.

--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://enterprisedb.com

+ It's impossible for everything to be true. +

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Joshua D. Drake 2011-07-12 16:03:09 Re: Full GUID support
Previous Message Magnus Hagander 2011-07-12 14:54:48 Re: remove README.mb.jp and README.mb.big5?