geodynamics / pylith_installer

Installer code for PyLith.
MIT License
7 stars 8 forks source link

Patch 1 #53

Closed ericlindsey closed 1 year ago

ericlindsey commented 1 year ago

Typos in the installation instructions - the docker environment has environment variables TOPSRC_DIR and TOPBUILD_DIR, but in the instructions they were written with the underscore in the wrong place. This made it hard to run the code by copy & pasting

baagaard-usgs commented 1 year ago

Thanks for reporting this. The documentation is correct. The Docker image is wrong. I believe I fixed this in the Docker build procedure and may just need to push the image to the container registry.

baagaard-usgs commented 1 year ago

@ericlindsey I just pushed new Docker images for the development environment. There are different Docker images for amd64 (Intel) versus arm64 (Apple) processors. I forgot to update the documentation after creating these images; the documentation is updated as well with instructions for using these the architecture-dependent images.