On 9/2/21 8:52 PM, Andrew Dunstan wrote:

On 7/22/21 3:49 AM, Erik Rijkers wrote:
Hi

Here are the 4 unchanged patches from v49, to which I added 2 patches,
which are small changes wrt usage of  'JsonIs'  versus  'IsJson'.

That should make the cfbot green again.


Apparently not, but I have rebased this and the sql/json function patch
set and incorporated your changes in both.


> [0001-SQL-JSON-functions-v50.patch]
> [0002-JSON_TABLE-v50.patch]
> [0003-JSON_TABLE-PLAN-DEFAULT-clause-v50.patch]
> [0004-JSON_TABLE-PLAN-clause-v50.patch]


These don't apply any more, could you have a look?

Thanks,

Erik Rijkers



(output from gcc 11.2:)

parse_jsontable.c: In function ‘makeStringConst’:
parse_jsontable.c:57:15: error: ‘union ValUnion’ has no member named ‘type’
   57 |         n->val.type = T_String;
      |               ^
parse_jsontable.c:58:16: error: ‘union ValUnion’ has no member named ‘val’; did you mean ‘ival’?
   58 |         n->val.val.str = str;
      |                ^~~
      |                ival
parse_jsontable.c: In function ‘transformJsonTable’:
parse_jsontable.c:714:61: error: ‘union ValUnion’ has no member named ‘type’
714 | castNode(A_Const, jt->common->pathspec)->val.type != T_String)
      |                                                             ^
parse_jsontable.c:721:65: error: ‘union ValUnion’ has no member named ‘val’; did you mean ‘ival’? 721 | rootPath = castNode(A_Const, jt->common->pathspec)->val.val.str;
      |                                                                 ^~~
| ival
make[3]: *** [parse_jsontable.o] Error 1
make[3]: *** Waiting for unfinished jobs....
make[2]: *** [parser-recursive] Error 2
make[2]: *** Waiting for unfinished jobs....
make[1]: *** [all-backend-recurse] Error 2
make: *** [all-src-recurse] Error 2
../../../src/Makefile.global:938: recipe for target 'parse_jsontable.o' failed
common.mk:39: recipe for target 'parser-recursive' failed
Makefile:42: recipe for target 'all-backend-recurse' failed
GNUmakefile:11: recipe for target 'all-src-recurse' failed



Reply via email to