Install build requirements for animate#177
Conversation
Adds `pip install -r animate/requirements-build.txt` step before building/installing animate. This is needed in the release container because there firedrake is installed directly from a pypi wheel and thus build requirements like Cython and a recent enough setuptools are not available. In the main container firedrake has been built locally and thus these are available already. Adding steps in both cases though for consistency. See mesh-adaptation/animate#241
|
@joewallwork : any idea why this PR hasn't triggered .github/workflows/docker_firedrake-parmmg_main.yml or .github/workflows/docker_firedrake-parmmg_release.yml ? I'm feeling I'm missing something obvious... |
For the release workflow, we do need to update the triggers in docs/.github/workflows/docker_firedrake-parmmg_release.yml Lines 7 to 20 in cdd8f3c to refer to the release version of the Dockerfile. I have no idea why the other one hasn't triggered. I've been having a lot of issues with GitHub Actions getting triggered in other repos recently. |
ad27b16 to
1953e60
Compare
52cb537 to
99cdbae
Compare
|
I was missing something obvious: some of the workflows had been "disabled due to inactivity (>60 days)" |
Adds
pip install -r animate/requirements-build.txtstep before building/installing animate. This is needed in the release container because there firedrake is installed directly from a pypi wheel and thus build requirements like Cython and a recent enough setuptools are not available. In the main container firedrake has been built locally and thus these are available already. Adding steps in both cases though for consistency.Addresses mesh-adaptation/animate#241