| From: | Manni Wood <manni(dot)wood(at)enterprisedb(dot)com> |
|---|---|
| To: | Jim Jones <jim(dot)jones(at)uni-muenster(dot)de> |
| Cc: | Nishant Sharma <nishant(dot)sharma(at)enterprisedb(dot)com>, Álvaro Herrera <alvherre(at)kurilemu(dot)de>, pgsql-hackers(at)postgresql(dot)org |
| Subject: | Re: [PATCH] Add pg_get_tablespace_ddl() function to reconstruct CREATE TABLESPACE statement |
| Date: | 2025-11-08 04:19:38 |
| Message-ID: | CAKWEB6ovQP+649hE9B=X+ZRbXR28C=3YwWwhD6a3aOBykYp8_A@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Fri, Nov 7, 2025 at 6:03 PM Jim Jones <jim(dot)jones(at)uni-muenster(dot)de> wrote:
>
>
> On 08/11/2025 00:38, Manni Wood wrote:
> > Alas, the build https://commitfest.postgresql.org/patch/6175/ <https://
> > commitfest.postgresql.org/patch/6175/> now fails, and I cannot reproduce
> > on my machine. Obviously there will be a v9...
>
> You forgot the declaration for build_tablespace_ddl_string[1]:
>
> ruleutils.c:13755:1: warning: no previous prototype for
> ‘build_tablespace_ddl_string’ [-Wmissing-prototypes]
> 13755 | build_tablespace_ddl_string(const Oid tspaceoid)
> | ^~~~~~~~~~~~~~~~~~~~~~~~~~~
>
> Best, Jim
>
> 1 - https://cirrus-ci.com/task/4855404196265984?logs=build#L1911
>
>
Thank you very much, Jim. Serves me right for looking at the error at the
end of the logs rather than the warning in the middle.
v9 is attached.
--
-- Manni Wood EDB: https://www.enterprisedb.com
| Attachment | Content-Type | Size |
|---|---|---|
| v9-0002-Adds-pg_get_tablespace_ddl-function.patch | text/x-patch | 14.8 KB |
| v9-0001-Supporting-changes-for-pg_get_tablespace_ddl-func.patch | text/x-patch | 9.0 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Manni Wood | 2025-11-08 04:46:28 | Re: [PATCH] Add pg_get_tablespace_ddl() function to reconstruct CREATE TABLESPACE statement |
| Previous Message | Bruce Momjian | 2025-11-08 02:54:07 | Re: Extra blank line in StrategyGetBuffer |