On Sun, 15 Jan 2023 at 09:39, Ajin Cherian <itsa...@gmail.com> wrote: > > On Fri, Jan 13, 2023 at 5:33 PM vignesh C <vignes...@gmail.com> wrote: > > Adding support for CREATE/ALTER/DROP Publication ddl deparsing. > > The attached v61 patch has the changes for the same. > > > > Hi Vignesh, > this doesn't seem to compile: > > gcc -std=gnu99 -Wall -Wmissing-prototypes -Wpointer-arith > -Wdeclaration-after-statement -Werror=vla -Wendif-labels > -Wmissing-format-attribute -Wformat-security -fno-strict-aliasing > -fwrapv -fexcess-precision=standard -g -ggdb -Og -g3 > -fno-omit-frame-pointer -I../../../src/include -D_GNU_SOURCE -c -o > ddl_deparse.o ddl_deparse.c > ddl_deparse.c: In function ‘deparse_PublicationObjects’: > ddl_deparse.c:8956:3: error: unknown type name ‘publication_rel’ > publication_rel *pub_rel = (publication_rel *) lfirst(lc1); > ^ > ddl_deparse.c:8956:31: error: ‘publication_rel’ undeclared (first use > in this function) > publication_rel *pub_rel = (publication_rel *) lfirst(lc1); > ^ > ddl_deparse.c:8956:31: note: each undeclared identifier is reported > only once for each function it appears in > ddl_deparse.c:8956:48: error: expected expression before ‘)’ token > publication_rel *pub_rel = (publication_rel *) lfirst(lc1);
I was able to apply all the patches and compile successfully, also cfbot as in [1] has compiled successfully. One thing I noticed is that structure is available at 006 patch, if you did not apply all the patch it might fail, we will take care of structure movement in the next version. [1] - https://cirrus-ci.com/task/5231672585617408 Regards, Vignesh
