Attention is currently required from: Angel Pons, Elyes Haouas, Martin L Roth.
Elyes Haouas has uploaded a new patch set (#5) to the change originally created by Martin L Roth. ( https://review.coreboot.org/c/coreboot/+/81024?usp=email )
The following approvals got outdated and were removed: Code-Review+1 by Angel Pons, Code-Review+2 by Elyes Haouas, Verified+1 by build bot (Jenkins)
Change subject: 3rdparty: Remove chromeec submodule ......................................................................
3rdparty: Remove chromeec submodule
The chromeec submodule is the largest submodule being pulled into the coreboot tree, at over 400MB. The main branch also contains the majority of these commits, so restricting it to a single branch still fetches over 350MB.
Because there is only a single mainboard directory that enables the build of the chromeec codebase by default, most people are fetching this repo for no reason.
Based on this, we're going to change the way that the chromeec submodule is used, fetching it the way we currently fetch external payloads. This gives us 2 large advantages: 1) Only builds that actually need the chromeec repo will pull it down. 2) Each board that wants to build the chromeec codebase can use a different commit, unlike submodules which all use the same "current" commit.
Signed-off-by: Martin Roth gaumless@gmail.com Change-Id: I357c4c9b506dd3817a308232446144ae889bc220 --- M .gitmodules D 3rdparty/chromeec 2 files changed, 0 insertions(+), 4 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/24/81024/5
-
Elyes Haouas (Code Review)