Skip to content

Conversation

@dragarcia
Copy link
Contributor

first iteration

@dragarcia dragarcia requested a review from darora March 1, 2022 01:36
Copy link
Contributor

@darora darora left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggest removing the flag enabling cassert ; let's get an rc build out

@darora
Copy link
Contributor

darora commented Mar 1, 2022

On second thoughts, let's leave it in, but only use this build for debugging - won't be releasing this for normal usage

@darora
Copy link
Contributor

darora commented Mar 1, 2022

In which case we can also use -O0 -g, since we're not pushing this for wider prod usage

@dragarcia dragarcia merged commit 44d5138 into develop Mar 1, 2022
@dragarcia dragarcia deleted the chore/enable_debug_symbols branch March 1, 2022 02:10
dragarcia added a commit that referenced this pull request Mar 21, 2022
darora pushed a commit that referenced this pull request Mar 21, 2022
hf pushed a commit that referenced this pull request Jan 23, 2023
* chore: enable debug symbols on PG compile

* bump to rc version

* use -O0 -g instead
hf pushed a commit that referenced this pull request Jan 23, 2023
damonrand pushed a commit to cepro/postgres that referenced this pull request Jun 15, 2025
* chore: enable debug symbols on PG compile

* bump to rc version

* use -O0 -g instead
damonrand pushed a commit to cepro/postgres that referenced this pull request Jun 15, 2025
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