search

crate / docker-crate

Source repository for the official CrateDB Docker image
https://registry.hub.docker.com/_/crate/
Apache License 2.0
49 stars 22 forks source link

Fix yum within CrateDB's docker image #176

Closed autophagy closed 4 years ago

autophagy commented 4 years ago

Summary of the changes / Why this is an improvement

Yum, the CentOS package manager, requires Python 2. In the course of installing Python 3.6, we symlink the default system Python from Python 2 to 3.

The shebang at the top of yum's main file is '!#/usr/bin/python', which now references the 3.6 installation. This commit replaces the shebang at the top of yum's script to point, instead, to '!#/usr/bin/python2'.

@MarkusH I'm not entirely sure how I feel about this. :scream: Would you have any other suggestions?

mfussenegger commented 4 years ago

I would also rather adapt crash to use the correct python instead of messing with the global /usr/bin/python