opensafely-core / stata-docker

Builds the opensafely stata docker image.
0 stars 0 forks source link

Switch to using a simpler-but-still-hacky way of invoking stata #6

Closed bloodearnest closed 3 years ago

bloodearnest commented 3 years ago

We create a wrapper script around the supplied script that writes a file if it successfully runs the script. We use the presence of this file as indication of success or failure, and exit appropriately.

We can then use the regular stata (without -b), which gives us a much simpler streaming stdout.

We also refactor the Dockerfile to our new approach, and add a very basic test suite