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

fix the error in mac os while building psycopg2 from source #42

Merged
merged 2 commits into from
Mar 23, 2023
Merged

fix the error in mac os while building psycopg2 from source #42

merged 2 commits into from
Mar 23, 2023

Conversation

charithmadhuranga
Copy link
Contributor

No need to build from source while psycopg2-binary already available.this fix most of the errors and unnecessary dependencies clashes happen when building from source.

… source while psycopg2-binary already available.
@freol35241
Copy link
Owner

Hi @charithmadhuranga, thank you for this PR. Can you please provide som more context about what errors you are seeing with the source build?

In general, the documentation of psycopg2 states the following:

The binary package is a practical choice for development and testing but in production it is advised to use the package built from sources.

So let's try to understand why you think this is necessary first.

@charithmadhuranga
Copy link
Contributor Author

hi freol35241, I was able to fix the error which cause at the integration installation time,by installing postgresql formula in homebrew before installing the ltss, brew install postgresql,this will install pkg_config ,psql as well as full postgresql server in the mac os ,which can be reduced if you are on development setup like me which simply using psycopg2-binary rather than the psycopg2.

@freol35241 freol35241 merged commit b592cbb into freol35241:master Mar 23, 2023
@freol35241
Copy link
Owner

Thanks @charithmadhuranga , I decided to merge this after all, to make life a bit simpler 🥳

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.

None yet

2 participants