Skip to content

Conversation

@pyramation
Copy link
Contributor

Summary

Fixes the Docker build to successfully compile all 4 PostgreSQL extensions (pgvector, PostGIS, pg_textsearch, pgsodium).

Changes:

  • Add perl, flex, bison build dependencies required by PostGIS (configure was failing with "Perl was not found")
  • Pin pg_textsearch to v0.2.0 and add version ARG for consistency
  • Patch pg_textsearch to add missing #include <math.h> (upstream bug causing build failure)

Review & Testing Checklist for Human

  • Verify the math.h patch is acceptable - The sed command patches an upstream bug in pg_textsearch. This is fragile and may need updating if upstream changes. Consider whether to report this upstream.
  • Run make test to verify all 4 extensions can be created:
    make test
    Expected output: all extensions OK after creating vector, postgis, pg_textsearch, and pgsodium extensions
  • Note pg_textsearch is prerelease - v0.2.0 shows a warning that it's not recommended for production use

Notes

  • pg_textsearch v0.2.0 displays: "This is prerelease software and should not be used in production"
  • The upstream math.h bug exists in both main branch and v0.2.0 tag of timescale/pg_textsearch

Link to Devin run: https://app.devin.ai/sessions/167cadea4ff447f098e983f2a5d8d66c
Requested by: Dan Lynch (@pyramation)

- Add perl, flex, bison build dependencies required by PostGIS
- Add PG_TEXTSEARCH_VERSION ARG for consistency
- Pin pg_textsearch to v0.2.0
- Fix missing math.h include in pg_textsearch (upstream bug)
@devin-ai-integration
Copy link

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@pyramation pyramation merged commit 5c75800 into main Jan 1, 2026
1 check failed
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

Successfully merging this pull request may close these issues.

2 participants