Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

pg17 not compiling yet #55

Open
luss opened this issue May 23, 2024 · 2 comments
Open

pg17 not compiling yet #55

luss opened this issue May 23, 2024 · 2 comments

Comments

@luss
Copy link
Contributor

luss commented May 23, 2024

I tagged 3.0beta1 and noticed and it looks like we need some #ifdef's to compile against pg17.

125:21: note: 'create_foreignscan_path' declared here
125 | extern ForeignPath *create_foreignscan_path(PlannerInfo *root, RelOptInfo *rel,
| ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
126 | PathTarget *target,
| ~~~~~~~~~~~~~~~~~~~
127 | double rows, Cost startup_cost, Cost total_cost,
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
128 | List *pathkeys,
| ~~~~~~~~~~~~~~~
129 | Relids required_outer,
| ~~~~~~~~~~~~~~~~~~~~~~
130 | Path *fdw_outerpath,
| ~~~~~~~~~~~~~~~~~~~~
131 | List *fdw_restrictinfo,
| ~~~~~~~~~~~~~~~~~~~~~~~
132 | List *fdw_private);
| ~~~~~~~~~~~~~~~~~
2 errors generated.
make: *** [/opt/pgbin-build/pgbin/comp20240523_0820/build/multicorn-pg17-3.0beta1-1-arm9/lib/postgresql/pgxs/src/makefiles/../../src/Makefile.global:1090: src/multicorn.bc] Error 1
[build@eqn-arm9 bld]$

@devrimgunduz
Copy link

Hi @luss ,

This fixes the builds:

d57b998

Any chance you can release a new tarball so that I can build the RPMs?

Thanks!

Regards, Devrim

@mfenniak
Copy link
Collaborator

Note that multicorn2 isn't currently running any tests against PG17. The test suite can be updated in flake.nix but (lightly) requires the release to be packaged in nixpkgs first.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants