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

chore: Windows: update to libpq-15.3 #442

Merged
merged 2 commits into from
Oct 9, 2023
Merged

chore: Windows: update to libpq-15.3 #442

merged 2 commits into from
Oct 9, 2023

Conversation

jeroen
Copy link
Member

@jeroen jeroen commented Oct 8, 2023

This also adds support for ARM, which CRAN is going to support soon.

@jeroen jeroen force-pushed the winlibs branch 2 times, most recently from 6a02e61 to 6c995a6 Compare October 8, 2023 13:38
@jeroen
Copy link
Member Author

jeroen commented Oct 8, 2023

I think this is good to merge. The Windows compiler warning is pre-existing and is triggered by your vendored boost, and is unrelated to this PR:

Found the following significant warnings:
  vendor/boost/intrusive/detail/slist_node.hpp:56:16: warning: pointer used after 'void operator delete(void*, std::size_t)' [-Wuse-after-free]
  vendor/boost/intrusive/detail/hook_traits.hpp:77:10: warning: pointer used after 'void operator delete(void*, std::size_t)' [-Wuse-after-free]

@krlmlr
Copy link
Member

krlmlr commented Oct 8, 2023

Good catch regarding this warning. Is this a g++12 bug? I could update the vendored headers to see if it goes away, but I wonder if an Rtools update is pending that will upgrade GCC.

@krlmlr krlmlr changed the title Windows: update to libpq-15.3 chore: Windows: update to libpq-15.3 Oct 9, 2023
@krlmlr krlmlr merged commit 3b99e0f into main Oct 9, 2023
15 of 16 checks passed
@krlmlr krlmlr deleted the winlibs branch October 9, 2023 18:08
@krlmlr
Copy link
Member

krlmlr commented Oct 9, 2023

Thanks. It's great to have these warnings appear in CI/CD too 😊 . The same warnings are seen on CRAN, so perhaps not worth holding our breath.

@jeroen
Copy link
Member Author

jeroen commented Oct 9, 2023

Yes I think it is a bug in gcc-12. It is fixed in gcc-13. I can suggest a hack if you like.

@krlmlr
Copy link
Member

krlmlr commented Oct 9, 2023

A hack would be great, thanks!

@jeroen
Copy link
Member Author

jeroen commented Oct 9, 2023

OK here you go: #443

krlmlr added a commit that referenced this pull request Mar 31, 2024
RPostgres 1.4.6

- Breaking change: `dbListObjects()` only allows `Id()` objects as `prefix` argument (@dpprdan, #390).

- Use `NULL` in favor of `NULL::text` when quoting strings and literals, to support JSON and other text-ish types. Fixes a regression introduced in #370 (#393, #425).

- `dbQuoteLiteral()` correctly quotes 64-bit integers from the bit64 package (of class `"integer64"`) (@karawoo, #435, #436).

- Breaking change: `dbListObjects()` only allows `Id()` objects as `prefix` argument (@dpprdan, #390).

- Windows: update to libpq-15.3 (#442).

- Upgrade boost to 1.81.0-1 to fix sprintf warnings (#417).

- Suppress warning in gcc-12 (#443).

- Tweak driver docs (@dpprdan, #433).

- Relicense as MIT.

- Close result set.

- Replace Rcpp by cpp11 (@Antonov548, #419).
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 9, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants