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

Package version above 14.1.0 does not install binaries #7

Open
louis-van-der-stam opened this issue Jun 7, 2022 · 4 comments
Open

Comments

@louis-van-der-stam
Copy link

When installing the latest (14.1.2) vesion from pypi.org the bin directory (python3.8/site-packages/postgresql/bin) remains empty. The same happens for 14.1.1. When installing 14.1.0 the bin directory is populated. Am I missing how to install or did something go wrong with packaging.
I'm using Python 3.8.13 on Linux

@aferdyp
Copy link

aferdyp commented Jun 26, 2022

The manylinux version currently being used by cibuildwheel is based on CentOS 7. The github action to build the wheel uses the "apt" version of Linux. The easiest fix here is to replace CIBW_BEFORE_ALL_LINUX: ${{ format('bash src/tools/install_pg.sh {0}', github.ref) }} with CIBW_BEFORE_ALL_LINUX: ${{ format('bash src/tools/install_pg_rpm.sh {0}', github.ref) }} in
wheels.yml

The downside here is that install_pg_rpm.sh does not include the build steps for postgis. The postgis build does not work quite well with CentOS 7. The dependencies are really old to get postgis 3.2 setup out of the box.

@louis-van-der-stam
Copy link
Author

Thanks for looking into this.

It reads like it is a problem when creating the package? If it is a problem when installing, then I'm lost with the explanation/instruction.

Can you help me out?

@aferdyp
Copy link

aferdyp commented Jun 27, 2022

It basically does come down to a problem with creating the package. I have raised a PR with the required change.

The process of building the wheel using cibuildwheel is dependent on using github actions. If you want to get a build for your own requirements (till the PR is approved), here's what you could possibly do -

  • Fork this repo
  • Make the change that's there in the PR
  • Create a new release with tag 14.4.x and publish
  • Check that the github action is running. When the build is done you should see an artifact zip. That should have the wheels you need.
  • (Unfortunately, till the time the PR is approved and a new release is triggered the wheels won't be in pypi)

@louis-van-der-stam
Copy link
Author

Thanks for explaining, I'm okay with using the somewhat older version. Just wanted to flag the problem and then understand the cause.
Again thanks for looking into this and making the package available!

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

No branches or pull requests

2 participants