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

Cannot build for PG16r1 #84

Closed
JamesInform opened this issue Sep 2, 2023 · 18 comments
Closed

Cannot build for PG16r1 #84

JamesInform opened this issue Sep 2, 2023 · 18 comments

Comments

@JamesInform
Copy link

I cannot build the compoent for PG16r1.

Are you planning to update the extension any time soon?

@mkgrgis
Copy link
Contributor

mkgrgis commented Sep 2, 2023

@JamesInform , please show the main compilation problems by your opinion. How many problems there are?

@JamesInform
Copy link
Author

Hi!

image

@mkgrgis
Copy link
Contributor

mkgrgis commented Sep 5, 2023

@JamesInform , this is make stop because anybody have never try to compile against PG16r1 ;-) Please add in

ifeq (,$(findstring $(MAJORVERSION), 11 12 13 14 15))
in your clone and ./test.sh How many tests will be failed? Maybe compilation will be impossible?

@mkgrgis
Copy link
Contributor

mkgrgis commented Sep 5, 2023

@JamesInform , you should also copy tests from 15.0 to your version identify directory, see https://github.com/pgspider/sqlite_fdw#tests and my extended draft https://github.com/mkgrgis/sqlite_fdw/tree/readme-fix-and-additions#tests.

@JamesInform
Copy link
Author

JamesInform commented Sep 5, 2023

Thanks. When I run ./test.sh I get:

image

Building throws an error:

image

@mkgrgis
Copy link
Contributor

mkgrgis commented Sep 5, 2023

@JamesInform , what was the method of Pg source getting in you case? git? wget ... tar.gz? Something other? Have you tested make check of PostgreSQL sources itself? Please also refer my draft about testing environment https://github.com/mkgrgis/sqlite_fdw/blob/readme-fix-and-additions/TESTING.md

@JamesInform
Copy link
Author

Are you able to compile the source against PG16rc1?

@mkgrgis
Copy link
Contributor

mkgrgis commented Sep 7, 2023

@JamesInform , I don't know about pgspider team, but I haven't tried to compile the source against PG16rc1. Anyway this need test file copying and analyse after resolving compilation problems if there are.

@JamesInform
Copy link
Author

Maybe you can try to compile.
It seems you are more familiar with testing and error identification than me.

@mkgrgis
Copy link
Contributor

mkgrgis commented Sep 8, 2023

It seems you are more familiar with testing and error identification than me.

I had have no testing experience at all some month before. In C projects usually somebody are welcome for questions and experiments. This is 2 sources of my testing experience. I have no experience to adopt some FDW to a new release.

@t-kataym
Copy link
Contributor

@JamesInform ,
We have been developing to support 16beta2 and will publish it by the end of this month.

@JamesInform
Copy link
Author

Great. Thanks for informing!

@devrimgunduz
Copy link

@t-kataym v16 is due tomorrow. What is the status of the patch?

Regards, Devrim

@t-kataym
Copy link
Contributor

Almost is done for v16beta2. We are in progress to confirm on v16.0 and will release in the next week.

@t-kataym t-kataym mentioned this issue Sep 21, 2023
@t-kataym
Copy link
Contributor

We have just released new sqlite_fdw supporting PostgreSQL16.0.

@devrimgunduz
Copy link

Thanks! I can confirm that it builds cleanly.

@mkgrgis
Copy link
Contributor

mkgrgis commented Sep 28, 2023

@devrimgunduz , could you please also give some advices around of #80 ?

@mkgrgis
Copy link
Contributor

mkgrgis commented May 22, 2024

@t-kataym I think this is solved and outdated issue after CI.yml using.

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

4 participants