Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix disabled dockerfiles #74161

Merged
merged 2 commits into from Jul 11, 2020
Merged

Fix disabled dockerfiles #74161

merged 2 commits into from Jul 11, 2020

Commits on Jul 8, 2020

  1. ci: disabled: riscv: minimise docker overlays 

    Suggested by @bjorn3

Every RUN command creates a new overlay on top of the image as of before
the RUN command. Using fewer RUN commands prevents intermediate overlays
(which in this case would have contained the entire Linux source tree).
    @tblah
    tblah committed Jul 8, 2020
    Configuration menu
    Copy the full SHA
    51b646e View commit details
    Browse the repository at this point in the history

  2. ci: fix context for disabled docker images 

    When the dockerfiles were moved into the host-x86_64 directory, paths
for COPY commands were updated with the new host-x86_64/ prefix. This
suggested that the intended context was src/ci/docker. However, the context
for disabled docker images was src/ci/docker/host-x86_64. This broke the new
paths and prevented src/ci/docker/scripts from being included in the
context at all.

This commit corrects this context allowing docker to find the files it
needs for COPY commands.
    @tblah
    tblah committed Jul 8, 2020
    Configuration menu
    Copy the full SHA
    d9fec59 View commit details
    Browse the repository at this point in the history