New issue
Advanced search Search tips

Issue 765670 link

Starred by 1 user

Issue metadata

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



Sign in to add a comment

Adjust cros_config_host to read config from stdin

Project Member Reported by sjg@chromium.org, Sep 15 2017

Issue description

In cros-unibuild.eclass we currently pipe the config into fdtget. We should add an option for this with cros_config_host also.

e.g. --stdin


 
Status: Started (was: Untriaged)
Project Member

Comment 2 by bugdroid1@chromium.org, Sep 19 2017

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

commit a8d43ac1491ddf4c6c9532c5d05052c3ad60330d
Author: Alec Thilenius <athilenius@chromium.org>
Date: Tue Sep 19 20:26:53 2017

chromeos-config: Added stdin loading to cros_config_host

Added the ability to pass - as the file path to cros_config_host and stream
the dtb config directly into stdin.

BUG= chromium:765670 
TEST=Ran all unit tests including newly added

Change-Id: I329d761bef89f7936fc1ff68ffdfee84a0935c76
Reviewed-on: https://chromium-review.googlesource.com/669302
Commit-Ready: Alec Thilenius <athilenius@chromium.org>
Tested-by: Alec Thilenius <athilenius@chromium.org>
Reviewed-by: Simon Glass <sjg@chromium.org>
Reviewed-by: Jason Clinton <jclinton@chromium.org>

[modify] https://crrev.com/a8d43ac1491ddf4c6c9532c5d05052c3ad60330d/chromeos-config/libcros_config/cros_config.cc
[modify] https://crrev.com/a8d43ac1491ddf4c6c9532c5d05052c3ad60330d/chromeos-config/cros_config_host_main.cc
[modify] https://crrev.com/a8d43ac1491ddf4c6c9532c5d05052c3ad60330d/chromeos-config/libcros_config/cros_config.h
[modify] https://crrev.com/a8d43ac1491ddf4c6c9532c5d05052c3ad60330d/chromeos-config/cros_config_host_main_unittest.cc

Status: Fixed (was: Started)

Sign in to add a comment