search

okfn / docker-ckan

Docker images and Docker Compose setup for CKAN [Not Maintained]
GNU Affero General Public License v3.0
81 stars 88 forks source link

Installing cryptography fails as it requires rust tools #66

Closed Zharktas closed 3 years ago

Zharktas commented 3 years ago

When ckan extension has PyOpenSSL as an requirement, the installation fails as compiling cryptography requires rust tools now. This happens for example in ckanext-harvest where latest master doesn't pass CI anymore.

Should rust tools be installed in the image or should the extension depending on cryptography install them themselves?

Apparently fixed already in #65, but the images in docker hub haven't been updated.

amercader commented 3 years ago

Thanks @Zharktas I could have sworn I pushed them but obviously didn't. Let me check

amercader commented 3 years ago

Images are up to date, but #65 was only applied to the datapusher image so it probably needs to be added to the 2.9 image as well. Will try to do it soon, also PRs welcome

Zharktas commented 3 years ago

Ah didn't notice that only datapusher image was updated with new dependencies, I'll submit a PR soon for this.

amercader commented 3 years ago

I added this change in 72b8fcc. Tests pass again: https://github.com/ckan/ckanext-harvest/actions/runs/602860133