diff options
| author | Hung-Te Lin <hungte@chromium.org> | 2018-11-30 23:29:27 +0800 |
|---|---|---|
| committer | chrome-bot <chrome-bot@chromium.org> | 2018-12-01 05:19:30 -0800 |
| commit | 73d3c0e3d2d170e9bd34dacdbbdaa5d52860dea2 (patch) | |
| tree | 60dd21424ff66a06b9fb0bafea5a674c35d64f63 | |
| parent | 979f8b906cfda7af57a4a526d352216eeafe1746 (diff) | |
| download | vboot-73d3c0e3d2d170e9bd34dacdbbdaa5d52860dea2.tar.gz | |
futility: updater: Skip applying white label for local build
For developers running a local build on white label models, currently
the chromeos-firmwareupdate will always fail if VPD `whitelabel_tag` is
set because the `keyset/` folder does not exist (which was created by
signer bot).
Developers in this case usually don't really care about which key to use
and will be happy with the default (DEV signed) keys, also the key
compatibility will be still checked later, so we can skip the white
label patching if no keyset folder, which would allow developers getting
same experience on WL and non-WL devices.
BUG=b:120268135
TEST=TEST=make futil; tests/futility/run_test_scripts.sh $(pwd)/build/futility
BRANCH=None
Change-Id: I3992301ff4c406096e11e1ae8129f2f68b2319b5
Signed-off-by: Hung-Te Lin <hungte@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/1356688
Reviewed-by: C Shapiro <shapiroc@chromium.org>
| -rw-r--r-- | futility/updater.c | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/futility/updater.c b/futility/updater.c index 2908d994..dcd37bf6 100644 --- a/futility/updater.c +++ b/futility/updater.c @@ -1850,7 +1850,14 @@ static int updater_setup_archive( cfg, arg, model->image, model->ec_image, model->pd_image); - if (model->is_white_label) { + if (model->is_white_label && !manifest->has_keyset) { + /* + * Developers running unsigned updaters (usually local build) + * won't be able match any white label tags. + */ + WARN("No keysets found - this is probably a local build of " + "unsigned firmware updater. Skip applying white label."); + } else if (model->is_white_label) { /* * It is fine to fail in updater_apply_white_label for factory * mode so we are not checking the return value; instead we |