Skip to content

Commit

Permalink
fix(docker): troute.nwm build
Browse files Browse the repository at this point in the history
Fix how `troute.nwm` is built. NOAA-OWP/t-route#784 removed `setup.py`, as a result the package should now be built with `build`.
  • Loading branch information
aaraney authored and robertbartel committed Jul 12, 2024
1 parent 400ad48 commit 7dcd654
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions docker/main/ngen/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -507,7 +507,8 @@ RUN if [ "${NGEN_WITH_ROUTING}" == "ON" ]; then \
&& python3 -m build . \
&& cp dist/*.whl ${WORKDIR}/t-route/wheels/ \
&& cd ${WORKDIR}/t-route/src/troute-nwm \
&& python3 setup.py bdist_wheel \
# troute-nwm doesn't use setup.py ... use build to make the wheel; see https://github.com/NOAA-OWP/t-route/pull/784 \
&& python3 -m build . \
&& cp dist/*.whl ${WORKDIR}/t-route/wheels/ ; \
fi

Expand Down Expand Up @@ -932,4 +933,4 @@ USER root
RUN pip install --no-cache-dir "git+https://github.com/noaa-owp/ngen-cal@${NGEN_CAL_COMMIT:-${NGEN_CAL_BRANCH}}#egg=ngen_cal&subdirectory=python/ngen_cal"

USER ${USER}
RUN chmod +x ${WORKDIR}/entrypoint.sh
RUN chmod +x ${WORKDIR}/entrypoint.sh

0 comments on commit 7dcd654

Please sign in to comment.