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

Fix plugin linking with as-needed #957

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

df7cb
Copy link

@df7cb df7cb commented May 29, 2019

On systems where the linker defaults to "as-needed" (Ubuntu), we need to
put -lpq at the end of the command line. SHLIB_LINK is the correct PGXS
flag for that, but it depends on using OBJS and MODULE_big.

Otherwise, using the plugin fails with "cannot find symbol PQSignal".

Extends the fix from #919.

On systems where the linker defaults to "as-needed" (Ubuntu), we need to
put -lpq at the end of the command line. SHLIB_LINK is the correct PGXS
flag for that, but it depends on using OBJS and MODULE_big.

Otherwise, using the plugin fails with "cannot find symbol PQSignal".
@wind39 wind39 added this to Triage in Roadmap May 29, 2019
@wind39 wind39 moved this from Needs Triage to High Priority in Roadmap May 29, 2019
@wind39 wind39 self-assigned this May 29, 2019
@wind39 wind39 self-requested a review May 29, 2019 21:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Roadmap
  
High Priority
Development

Successfully merging this pull request may close these issues.

None yet

2 participants