-
Notifications
You must be signed in to change notification settings - Fork 83
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
[MacOS] warning: passing arguments to a function without a prototype is deprecated in all versions of C and is not supported in C2x [-Wdeprecated-non-prototype] #81
Comments
@manitgupta Thanks for the report! Could you check out the current version (v4) and see whether that reproduces the same issue? |
After updating to pg_query_go v4, I am seeing some compile issues due to changed interfaces. I am not that well versed in this library, so posting the errors here (please bear with me) - Looks like the
While the older struct was -
Specifically, the data type of Link to code in the original repo -
|
That's correct - this is due to an upstream change in the Postgres node structs, which pg_query bases the parse tree objects on. I unfortunately don't have the capacity to help with changes on the Google HarbourBridge repository that uses pg_query_go here due to time constraints (in addition to not being familiar with that code base). My recommendation would be to work with the original authors of the HarbourBridge code to adapt it to the new parse node definitions from Postgres 15. I imagine this would be a goal anyway to support Postgres 15 syntax in HarbourBridge. |
I'm seeing these same warnings using v4 in sqlc. I'm on macOS 13.2. Inside of the sqlc repository, you can reproduce by doing the following:
|
Thanks for flagging this also occurs on v4. We'll see if that's an easy fix, might just be a matter of turning off the warning. /cc @msepga |
I'm seeing the same warnings as @kyleconroy when using sqlc. |
This fixes the warnings that arise in #81
I believe this is now fixed in #83 |
I started seeing the following warnings recently while building the upstream package of
pg_query_go
when building on MacOS -Could you please help me understand the issue? I see that the
sprintf
issue would is already logged on #67 and will be fixed in the subsequent release. Not sure about thepassing arguments to a function..
The text was updated successfully, but these errors were encountered: