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

Use our own memory area for editing libpq static error messages #903

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

Bodobolero
Copy link
Contributor

see segmentation fault described in neondatabase/neon#10706

@dimitri dimitri changed the title fix https://github.com/dimitri/pgcopydb/issues/901 Use our own memory area for editing libpq static error messages Feb 19, 2025
@dimitri
Copy link
Owner

dimitri commented Feb 24, 2025

Seems editing with the GitHub web UI makes it hard to pass our CI indentation test. At the minute I need to refresh my dev environment (Docker and macOS issues, re-install Citus indentation tooling). Will have another try later this week.

@Bodobolero
Copy link
Contributor Author

Bodobolero commented Feb 24, 2025

Seems editing with the GitHub web UI makes it hard to pass our CI indentation test. At the minute I need to refresh my dev environment (Docker and macOS issues, re-install Citus indentation tooling). Will have another try later this week.

I couldn't get your Citus indentation tooling to run on my Mac. If you find a solution pls let me know. That is why I couldn't really fix the style problems myself, since your GitHub workflow doesn't output what it doesn't like, either.

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.

3 participants