Skip to content

Commit

Permalink
Update RTI dockerfile to match file layout change
Browse files Browse the repository at this point in the history
This could harm reproducibility and could increase the size of the RTI
image. However, reproducibility is actually not much worse than the
status quo which was still copying whole directories. Also, the amount
of extra stuff copied is small. In the long run this will keep us from
running into further problems as we continue to refactor this repo
without testing the dockerized RTI build in CI (I am assuming that it
is not worthwhile to add a test for it in CI)
  • Loading branch information
petervdonovan committed Mar 22, 2024
1 parent 75fcdba commit 8158f7c
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions core/federated/RTI/rti.Dockerfile
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
# Docker file for building the image of the rti
FROM alpine:latest
COPY core /lingua-franca/core
COPY include /lingua-franca/include
COPY . /lingua-franca
WORKDIR /lingua-franca/core/federated/RTI
RUN set -ex && apk add --no-cache gcc musl-dev cmake make && \
mkdir container && \
Expand Down

0 comments on commit 8158f7c

Please sign in to comment.