| From: | Bruce Momjian <bruce(at)momjian(dot)us> |
|---|---|
| To: | Robert Haas <robertmhaas(at)gmail(dot)com> |
| Cc: | pgsql-hackers(at)postgresql(dot)org |
| Subject: | Re: pg_dump -X |
| Date: | 2011-03-11 03:36:55 |
| Message-ID: | 201103110336.p2B3atU16338@momjian.us |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Robert Haas wrote:
> Back in 2006, we have this commit:
>
> commit 2b25e1169f44368c120931787628d51731b5cc8c
> Author: Peter Eisentraut <peter_e(at)gmx(dot)net>
> Date: Sat Oct 7 20:59:05 2006 +0000
>
> The -X option in pg_dump was supposed to be a workaround for the lack of
> portable long options. But we have had portable long options for a long
> time now, so this is obsolete. Now people have added options which *only*
> work with -X but not as regular long option, so I'm putting a stop to this:
> -X is deprecated; it still works, but it has been removed from the
> documentation, and please don't add more of them.
>
> Since then, two additional -X options have crept in, doubtless due to
> mimicry of the existing options without examination of the commit
> logs. I think we should either (a) remove the -X option altogether or
> (b) change the comment so that it clearly states the same message that
> appears here in the commit log, namely, that no new -X options are to
> be created. The existing comment says that -X is deprecated, but that
> doesn't make it entirely 100% clear that the code isn't intended to be
> further updated, at least judging by the results.
Code comment added with attached, applied patch.
--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://enterprisedb.com
+ It's impossible for everything to be true. +
| Attachment | Content-Type | Size |
|---|---|---|
| /rtmp/pg_dump.diff | text/x-diff | 790 bytes |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Robert Haas | 2011-03-11 03:46:47 | Re: pg_dump -X |
| Previous Message | Bruce Momjian | 2011-03-11 03:28:01 | Re: psql expanded auto |