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

[ CPPFLAGS] Switch to USE_ASSERT_CHECKING #1826

Merged

Conversation

SamuelMarks
Copy link
Contributor

@SamuelMarks SamuelMarks commented Aug 23, 2024

[cargo-pgrx/src/command/init.rs] Use USE_ASSERT_CHECKING over MEMORY_CONTEXT_CHECKING & CLOBBER_FREED_MEMORY to avoid symbol redefinition warnings from postgres @ src/include/pg_config_manual.h

Related: My patch to PostgreSQL's pgsql-hackers mailing-list https://www.postgresql.org/message-id/CAMfPbcYq94UKjAr_R5YOPXWPNC9Sx9AOa6fS4Fio6tobEsmf5g%40mail.gmail.com

…RY_CONTEXT_CHECKING` & `CLOBBER_FREED_MEMORY` to avoid symbol redefinition warnings from postgres @ src/include/pg_config_manual.h
@workingjubilee workingjubilee merged commit 7107b51 into pgcentralfoundation:develop Aug 24, 2024
14 checks passed
eeeebbbbrrrr added a commit that referenced this pull request Sep 6, 2024
Welcome to pgrx v0.12.2. This is a minor release that fixes a few bugs,
improves compilation times with `cargo pgrx run/install/test`, and adds
a few more Postgres headers.

As usual, please `cargo install cargo-pgrx --version 0.12.2 --locked`.
Then you can run `cargo pgrx upgrade` in your extension crate's root to
update its dependencies.

## What's Changed

* [ `CPPFLAGS`] Switch to `USE_ASSERT_CHECKING` by @SamuelMarks in
#1826
* Add UnboxDatum for Range<T> by @mhov in
#1827
* Moved Sized from FromDatum methods to trait by @YohDeadfall in
#1831
* Used SPI result type for cursor API by @YohDeadfall in
#1836
* always compile `pgrx_embed_*` without opts by @eeeebbbbrrrr in
#1838
* Fixed out of bounds reads for open_cursor by @YohDeadfall in
#1837
* Include `access/visibilitymap.h` and `utils/tuplestore.h` by
@eeeebbbbrrrr in #1841

## New Contributors

Thanks to these folks for their first-time contributions -- it's greatly
appreciated!

* @SamuelMarks made their first contribution in
#1826
* @YohDeadfall made their first contribution in
#1831

**Full Changelog**:
v0.12.1...v0.12.2
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