-
Notifications
You must be signed in to change notification settings - Fork 599
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
Docker image build fails for distutils #320
Comments
I don't use Docker so this hasn't been updated for a while. Since we now use the pre-built pycolmap available on pypi, we don't need to depend on the COLMAP Docker image - any other will do. |
FYI colmap has begun pushing new images (the previous image was 4 years old and based on colmap 3.6-dev.3). The new image does not have distutils.cmd installed. (distutils is actually deprecated now and was removed in python 3.12). This can be fixed by installing python3.8-distutils: apt-get update
apt-get install -y python3.8-distutils However, now colmap/colmap:latest actually has python 3.10 installed. As such, I think these lines could just be removed from the Dockerfile. |
When removing the lines, it is necessary to install Building the docker container without Lines 5-10 and with line 4 changed to I created pull request #330 to update the Dockerfile. |
I got this from main branch, is this due some python dependencies that doesn't anymore work?
The text was updated successfully, but these errors were encountered: