Skip to content

match documentation to upstream C library #2

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

Merged
merged 2 commits into from
Jul 2, 2019
Merged

Conversation

bergey
Copy link
Contributor

@bergey bergey commented Jul 1, 2019

A couple of mismatches I noticed between docs on the Haskell bindings & on upstream libpq. I'm not sure which versions of libpq this library tries to support. If you only support libpq that still have upstream support, I would prefer to drop the warning about ntuples entirely.

previously it had the doc string for `errorMessage`
@phadej
Copy link
Collaborator

phadej commented Jul 1, 2019

debian old-stable seems to have libpq-9.4, so I'm fine with dropping ntuples warning.

@phadej
Copy link
Collaborator

phadej commented Jul 1, 2019

In fact, we have

  if flag(use-pkg-config)
    Pkgconfig-depends:   libpq >=9.3

The parenthetical has been included in libpq docs since 9.3.  This
library already requires libpq >= 9.3.
@bergey
Copy link
Contributor Author

bergey commented Jul 2, 2019

Thanks! I updated this branch, removing the old warning.

@phadej phadej merged commit 485341c into haskellari:master Jul 2, 2019
@bergey bergey deleted the docs branch July 3, 2019 14:30
robx added a commit to robx/postgresql-libpq that referenced this pull request Mar 8, 2023
Merge postgresql-libpq upstream as of 2022-12-29
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