postgresql-wheel icon indicating copy to clipboard operation
postgresql-wheel copied to clipboard

Package version above 14.1.0 does not install binaries

Open louis-van-der-stam opened this issue 3 years ago • 4 comments

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

louis-van-der-stam avatar Jun 07 '22 14:06 louis-van-der-stam

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.

aferdyp avatar Jun 26 '22 09:06 aferdyp

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?

louis-van-der-stam avatar Jun 27 '22 07:06 louis-van-der-stam

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)

aferdyp avatar Jun 27 '22 07:06 aferdyp

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!

louis-van-der-stam avatar Jun 27 '22 07:06 louis-van-der-stam