<p>Nico Huber <strong>merged</strong> this change.</p><p><a href="https://review.coreboot.org/29455">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  build bot (Jenkins): Verified
  Martin Roth: Looks good to me, approved

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">coreboot-sdk: Build Clang first in a separate step<br><br>If we build one of the `all*` targets, build Clang first. Compiling<br>Clang (just for the host arch, I assume) takes more than half of the<br>time of the default build. When run as a separate step, we can make<br>use of Docker's cache if any step after Clang fails.<br><br>Change-Id: If67b458cde656f1dc6774215f6a575a48d12b797<br>Signed-off-by: Nico Huber <nico.h@gmx.de><br>Reviewed-on: https://review.coreboot.org/29455<br>Tested-by: build bot (Jenkins) <no-reply@coreboot.org><br>Reviewed-by: Martin Roth <martinroth@google.com><br>---<br>M util/docker/coreboot-sdk/Dockerfile<br>1 file changed, 12 insertions(+), 5 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/util/docker/coreboot-sdk/Dockerfile b/util/docker/coreboot-sdk/Dockerfile</span><br><span>index bdfbf4a..739e0d1 100644</span><br><span>--- a/util/docker/coreboot-sdk/Dockerfile</span><br><span>+++ b/util/docker/coreboot-sdk/Dockerfile</span><br><span>@@ -74,12 +74,19 @@</span><br><span> RUN \</span><br><span>         cd /root && \</span><br><span>        git clone https://review.coreboot.org/coreboot && \</span><br><span style="color: hsl(0, 100%, 40%);">-     cd coreboot/util/crossgcc && \</span><br><span style="color: hsl(0, 100%, 40%);">-  git checkout {{DOCKER_COMMIT}} && \</span><br><span style="color: hsl(0, 100%, 40%);">-     make {{CROSSGCC_PARAM}} \</span><br><span style="color: hsl(120, 100%, 40%);">+     cd coreboot && \</span><br><span style="color: hsl(120, 100%, 40%);">+      git checkout {{DOCKER_COMMIT}}</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+RUN \</span><br><span style="color: hsl(120, 100%, 40%);">+       if echo {{CROSSGCC_PARAM}} | grep -q ^all; then \</span><br><span style="color: hsl(120, 100%, 40%);">+             make -C/root/coreboot/util/crossgcc/ build_clang \</span><br><span style="color: hsl(120, 100%, 40%);">+                    BUILD_LANGUAGES=c,ada CPUS=$(nproc) DEST=/opt/xgcc; \</span><br><span style="color: hsl(120, 100%, 40%);">+ fi</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+RUN \</span><br><span style="color: hsl(120, 100%, 40%);">+   make -C/root/coreboot/util/crossgcc/ {{CROSSGCC_PARAM}} \</span><br><span>            BUILD_LANGUAGES=c,ada CPUS=$(nproc) DEST=/opt/xgcc && \</span><br><span style="color: hsl(0, 100%, 40%);">- cd /root && \</span><br><span style="color: hsl(0, 100%, 40%);">-   rm -rf coreboot</span><br><span style="color: hsl(120, 100%, 40%);">+       rm -rf /root/coreboot</span><br><span> </span><br><span> RUN mkdir /home/coreboot/.ccache && \</span><br><span>   chown coreboot:coreboot /home/coreboot/.ccache && \</span><br><span></span><br></pre><p>To view, visit <a href="https://review.coreboot.org/29455">change 29455</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://review.coreboot.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://review.coreboot.org/29455"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: coreboot </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: merged </div>
<div style="display:none"> Gerrit-Change-Id: If67b458cde656f1dc6774215f6a575a48d12b797 </div>
<div style="display:none"> Gerrit-Change-Number: 29455 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Nico Huber <nico.h@gmx.de> </div>
<div style="display:none"> Gerrit-Reviewer: Arthur Heymans <arthur@aheymans.xyz> </div>
<div style="display:none"> Gerrit-Reviewer: Martin Roth <martinroth@google.com> </div>
<div style="display:none"> Gerrit-Reviewer: Nico Huber <nico.h@gmx.de> </div>
<div style="display:none"> Gerrit-Reviewer: Patrick Georgi <pgeorgi@google.com> </div>
<div style="display:none"> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> </div>