New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 777726 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Nov 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Feature



Sign in to add a comment

Replace unibuild logic in chromeos-ec with something simpler

Project Member Reported by sjg@chromium.org, Oct 24 2017

Issue description

We have decided that the master configuration can be different between ToT and a branch. This allows us to simplify the chromeos-ec ebuild.

To do:
- Update reef master config on ToT so that it does not build 
- Do nothing with firmware branch: for now it has separate builders so is WAI. Before we turn down those builds we will need to do more work
- Use cros_config_host_py get-firmware-build-targets ec
- Add 'cros_config_host_py get-firmware-build-targets cr50' to handle that
- Simplify chromeos-ec build to remove additional checks

 
Project Member

Comment 1 by bugdroid1@chromium.org, Oct 29 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform2/+/6728fb2516c2d909751bb7e28dbc7126947631c6

commit 6728fb2516c2d909751bb7e28dbc7126947631c6
Author: Simon Glass <sjg@chromium.org>
Date: Sun Oct 29 01:08:47 2017

chromeos-config: Add cr50 to the binding

This is another type of firmware that we can build. Add support for it ihe
host tool and validator.

BUG= chromium:777726 
TEST=FEATURES=test emerge-reef-uni --nodeps chromeos-config-tools

Change-Id: I067c09993ba54f05d9fda5625f49c9aa62c3ff36
Signed-off-by: Simon Glass <sjg@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/734441
Reviewed-by: Jason Clinton <jclinton@chromium.org>

[modify] https://crrev.com/6728fb2516c2d909751bb7e28dbc7126947631c6/chromeos-config/README.md
[modify] https://crrev.com/6728fb2516c2d909751bb7e28dbc7126947631c6/chromeos-config/libcros_config_host/libcros_config_host.py
[modify] https://crrev.com/6728fb2516c2d909751bb7e28dbc7126947631c6/chromeos-config/validate_config.py

Project Member

Comment 2 by bugdroid1@chromium.org, Nov 1 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/4b224417307a2f52c17760ec65e58b7f2cac37c6

commit 4b224417307a2f52c17760ec65e58b7f2cac37c6
Author: Simon Glass <sjg@chromium.org>
Date: Wed Nov 01 18:26:01 2017

chromeos-ec: Simplify the ebuild for unibuild

This is currently quite complicated because we cannot rely on the master
configuration to tell us what to do. We're going to fix that, which allows
the ebuild to be simplified.

Update the ebuild to use cros_config_host_py and do exactly what it is
told.

BUG= chromium:777726 
TEST=emerge-reef-uni chromeos-ec
emerge-coral chromeos-ec
See that it builds the expected targets
Change-Id: I8b87df8a3cfe095b3f16084479646e9eec730633
Signed-off-by: Simon Glass <sjg@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/733090
Reviewed-by: Randall Spangler <rspangler@chromium.org>

[modify] https://crrev.com/4b224417307a2f52c17760ec65e58b7f2cac37c6/eclass/cros-ec-board.eclass
[modify] https://crrev.com/4b224417307a2f52c17760ec65e58b7f2cac37c6/chromeos-base/chromeos-ec/chromeos-ec-9999.ebuild

Project Member

Comment 3 by bugdroid1@chromium.org, Nov 1 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/91773c6fad5b7473f3444661cac7459b9b426bc4

commit 91773c6fad5b7473f3444661cac7459b9b426bc4
Author: Simon Glass <sjg@chromium.org>
Date: Wed Nov 01 23:47:19 2017

coreboot: Simplify the ebuild for unibuild

This is currently quite complicated because we cannot rely on the master
configuration to tell us what to do. We're going to fix that, which allows
the ebuild to be simplified.

Update the ebuild to use cros_config_host_py and do exactly what it is
told.

BUG= chromium:777726 
TEST=emerge-reef-uni coreboot chromeos-bootimage
See that it builds the expected target

Change-Id: I450a8632ac462d33de282936aa98fd8d357293f5
Signed-off-by: Simon Glass <sjg@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/742380
Reviewed-by: Jason Clinton <jclinton@chromium.org>

[modify] https://crrev.com/91773c6fad5b7473f3444661cac7459b9b426bc4/sys-boot/coreboot/coreboot-9999.ebuild

Comment 4 by sjg@chromium.org, Nov 4 2017

Labels: Unibuild

Comment 5 by sjg@chromium.org, Nov 6 2017

Status: Fixed (was: Available)
It is back to being simple now.

Sign in to add a comment