Jonathan Neuschäfer has posted comments on this change. ( https://review.coreboot.org/29444 )
Change subject: Linuxboot: gitignore build directory
......................................................................
Patch Set 1:
(1 comment)
Other payloads' source/build directories are so far ignored in the top-level .gitgnore
https://review.coreboot.org/#/c/29444/1/payloads/external/LinuxBoot/.gitign…
File payloads/external/LinuxBoot/.gitignore:
https://review.coreboot.org/#/c/29444/1/payloads/external/LinuxBoot/.gitign…
PS1, Line 1: linuxboot/*
The star (*) is probably not necessary here
--
To view, visit https://review.coreboot.org/29444
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic11311e2de544a334043a55eda42643d9df3374d
Gerrit-Change-Number: 29444
Gerrit-PatchSet: 1
Gerrit-Owner: Marcello Sylvester Bauer <info(a)marcellobauer.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Jonathan Neuschäfer <j.neuschaefer(a)gmx.net>
Gerrit-Comment-Date: Sun, 04 Nov 2018 20:07:36 +0000
Gerrit-HasComments: Yes
Gerrit-HasLabels: No
Nico Huber has posted comments on this change. ( https://review.coreboot.org/29451 )
Change subject: util/abuild: Use absolute paths for (failed|passed)_boards
......................................................................
Patch Set 1: Code-Review-2
needs refinement, `realpath` seems to be non-standard and ${TARGET} has
to be created first
--
To view, visit https://review.coreboot.org/29451
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ief00e28e09a4714b0106c35ad4f4e9c696f6aad4
Gerrit-Change-Number: 29451
Gerrit-PatchSet: 1
Gerrit-Owner: Nico Huber <nico.h(a)gmx.de>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Nico Huber <nico.h(a)gmx.de>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Sun, 04 Nov 2018 20:02:27 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: Yes
Nico Huber has removed build bot (Jenkins) from this change. ( https://review.coreboot.org/29457 )
Change subject: util/docker: Use common .ccache for local builds
......................................................................
Removed reviewer build bot (Jenkins) with the following votes:
* Verified+1 by build bot (Jenkins) <no-reply(a)coreboot.org>
--
To view, visit https://review.coreboot.org/29457
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: deleteReviewer
Gerrit-Change-Id: I7d0bdb3861ac56361cacfa74aaf8b45c4f135e5c
Gerrit-Change-Number: 29457
Gerrit-PatchSet: 1
Gerrit-Owner: Nico Huber <nico.h(a)gmx.de>
Nico Huber has uploaded a new patch set (#2). ( https://review.coreboot.org/29454 )
Change subject: coreboot-sdk: Prefer gnat package over gnat-6
......................................................................
coreboot-sdk: Prefer gnat package over gnat-6
The choice of `gnat-6` was originally an optimization because the meta-
package `gnat` installs not only the current GNAT version but also other
unwanted (and hard to explain) dependencies. Later it was necessary
because GCC 8 couldn't compile our older crossgcc.
Now that we switched crossgcc to GCC 8.1, `gnat` should be fine.
Change-Id: Ica8a1f9d6d71a74ffc4ec76aa0cfbe4b604cde1b
Signed-off-by: Nico Huber <nico.h(a)gmx.de>
---
M util/docker/coreboot-sdk/Dockerfile
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/54/29454/2
--
To view, visit https://review.coreboot.org/29454
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ica8a1f9d6d71a74ffc4ec76aa0cfbe4b604cde1b
Gerrit-Change-Number: 29454
Gerrit-PatchSet: 2
Gerrit-Owner: Nico Huber <nico.h(a)gmx.de>
Nico Huber has uploaded this change for review. ( https://review.coreboot.org/29455
Change subject: coreboot-sdk: Build Clang first in a separate step
......................................................................
coreboot-sdk: Build Clang first in a separate step
If we build one of the `all*` targets, build Clang first. Compiling
Clang (just for the host arch, I assume) takes more than half of the
time of the default build. When run as a separate step, we can make
use of Docker's cache if any step after Clang fails.
Change-Id: If67b458cde656f1dc6774215f6a575a48d12b797
Signed-off-by: Nico Huber <nico.h(a)gmx.de>
---
M util/docker/coreboot-sdk/Dockerfile
1 file changed, 12 insertions(+), 5 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/55/29455/1
diff --git a/util/docker/coreboot-sdk/Dockerfile b/util/docker/coreboot-sdk/Dockerfile
index bdfbf4a..739e0d1 100644
--- a/util/docker/coreboot-sdk/Dockerfile
+++ b/util/docker/coreboot-sdk/Dockerfile
@@ -74,12 +74,19 @@
RUN \
cd /root && \
git clone https://review.coreboot.org/coreboot && \
- cd coreboot/util/crossgcc && \
- git checkout {{DOCKER_COMMIT}} && \
- make {{CROSSGCC_PARAM}} \
+ cd coreboot && \
+ git checkout {{DOCKER_COMMIT}}
+
+RUN \
+ if echo {{CROSSGCC_PARAM}} | grep -q ^all; then \
+ make -C/root/coreboot/util/crossgcc/ build_clang \
+ BUILD_LANGUAGES=c,ada CPUS=$(nproc) DEST=/opt/xgcc; \
+ fi
+
+RUN \
+ make -C/root/coreboot/util/crossgcc/ {{CROSSGCC_PARAM}} \
BUILD_LANGUAGES=c,ada CPUS=$(nproc) DEST=/opt/xgcc && \
- cd /root && \
- rm -rf coreboot
+ rm -rf /root/coreboot
RUN mkdir /home/coreboot/.ccache && \
chown coreboot:coreboot /home/coreboot/.ccache && \
--
To view, visit https://review.coreboot.org/29455
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: If67b458cde656f1dc6774215f6a575a48d12b797
Gerrit-Change-Number: 29455
Gerrit-PatchSet: 1
Gerrit-Owner: Nico Huber <nico.h(a)gmx.de>
Nico Huber has uploaded this change for review. ( https://review.coreboot.org/29454
Change subject: coreboot-sdk: Prever gnat package over gnat-6
......................................................................
coreboot-sdk: Prever gnat package over gnat-6
The choice of `gnat-6` was originally an optimization because the meta-
package `gnat` installs not only the current GNAT version but also other
unwanted (and hard to explain) dependencies. Later it was necessary
because GCC 8 couldn't compile our older crossgcc.
Now that we switched crossgcc to GCC 8.1, `gnat` should be fine.
Change-Id: Ica8a1f9d6d71a74ffc4ec76aa0cfbe4b604cde1b
Signed-off-by: Nico Huber <nico.h(a)gmx.de>
---
M util/docker/coreboot-sdk/Dockerfile
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/54/29454/1
diff --git a/util/docker/coreboot-sdk/Dockerfile b/util/docker/coreboot-sdk/Dockerfile
index 44682f8..bdfbf4a 100644
--- a/util/docker/coreboot-sdk/Dockerfile
+++ b/util/docker/coreboot-sdk/Dockerfile
@@ -34,7 +34,7 @@
gawk \
gcc \
git \
- gnat-6 \
+ gnat \
graphviz \
libfreetype6-dev \
libftdi-dev \
--
To view, visit https://review.coreboot.org/29454
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ica8a1f9d6d71a74ffc4ec76aa0cfbe4b604cde1b
Gerrit-Change-Number: 29454
Gerrit-PatchSet: 1
Gerrit-Owner: Nico Huber <nico.h(a)gmx.de>