| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
. and .. are special names for the current and parent directories,
respectively. Thus, we should not use these names when formulating
filenames for testing.
BUG=chromium:1176487
BRANCH=none
TEST=zmake testall
Signed-off-by: Jack Rosenthal <jrosenth@chromium.org>
Change-Id: I2f16cb960878aa8a61f77ca1d074e0bac59bca3d
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2685403
Commit-Queue: Simon Glass <sjg@chromium.org>
Reviewed-by: Simon Glass <sjg@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This will be used to read the version from a non-CrOS checked-out
zephyr base.
BUG=b:179813084
BRANCH=none
TEST=provided unit tests
Signed-off-by: Jack Rosenthal <jrosenth@chromium.org>
Change-Id: Id6c3bdebbc5273df767f23115f00f2dac794e3a7
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2685474
Reviewed-by: Simon Glass <sjg@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The timing in the CQ seems rather inconsistent for this test.
Hypothesis complains about this:
> Unreliable test timings! On an initial run, this test took
> 1456.42ms, which exceeded the deadline of 1000.00ms, but on a
> subsequent run it took 183.34 ms, which did not. If you expect this
> sort of variability in your test timings, consider turning deadlines
> off for this test by setting deadline=None.
So, I'm setting deadline=None to see if it helps.
BUG=chromium:1175647
BRANCH=none
TEST=CQ de-flakes?
Signed-off-by: Jack Rosenthal <jrosenth@chromium.org>
Change-Id: I09bffa6d00ea6b5457c28d8eb9003e7bf077e9b3
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2682838
Reviewed-by: Yuval Peress <peress@chromium.org>
Reviewed-by: Simon Glass <sjg@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Allow board-specific DTS overlays to be placed at
zephyr/dts/board-overlays/${BOARD}.dts in any module. Zmake will find
the file and append it to the DTC_OVERLAY_FILE list when discovered.
BUG=b:179491802
BRANCH=none
TEST=provided unit test passes
Signed-off-by: Jack Rosenthal <jrosenth@chromium.org>
Change-Id: I53d4a04b6056689696b4615b5998937dbcf801c4
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2678795
Reviewed-by: Simon Glass <sjg@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The previous changes to the zmake build system made the
incorrect assumption that the build directory exists already
as such, running `zmake configure -B $build_dir` with a new
non-initialized director would fail since the
resolve_build_dir function would fail to recognize it as a
valid build directory.
The new logic also allows for passing directories which do
not yet exist.
BRANCH=none
BUG=none
TEST=pytest zephyr/zmake/tests/test_util.py
TEST=zmake configure -B /tmp/z/pos \
zephyr/projects/posix-ec/ \
&& zmake -l DEBUG build /tmp/z/pos
Signed-off-by: Yuval Peress <peress@chromium.org>
Change-Id: I754d4545f8d98cc65270b8d50f327cbb04ce3c87
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2678927
Tested-by: Simon Glass <sjg@chromium.org>
Reviewed-by: Jack Rosenthal <jrosenth@chromium.org>
Reviewed-by: Simon Glass <sjg@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Allows zmake to use the default platform/ec/build directory
when running the build/test subcommand. Developers can now
simply point to the actual projects without having to worry
about where the build is being done.
BRANCH=none
BUG=none
TEST=Added/modified unit tests
TEST=zmake configure zephyr/test/system && \
zmake build zephyr/test/system && \
zmake test zephyr/test/system && \
echo SUCCESS
TEST=rm -fr build/zephyr/test/system ; \
zmake build zephyr/test/system || echo SUCCESS
Signed-off-by: Yuval Peress <peress@chromium.org>
Change-Id: Id830f8dffc30f641fff0cd5ff50d30b2efedeece
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2665072
Reviewed-by: Simon Glass <sjg@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Allow zmake to create a default build directory when running zmake
configure. This provides a similar behavior to platform/ec's make
system. Projects will be placed under platform/ec/build/zephyr/...
BRANCH=none
BUG=none
TEST=Added unit test
TEST=zmake configure -b zephyr/projects/volteer
Signed-off-by: Yuval Peress <peress@chromium.org>
Change-Id: Icabed1b77cf5876a75898322d39806237e05d520
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2665071
Reviewed-by: Simon Glass <sjg@chromium.org>
Reviewed-by: Jack Rosenthal <jrosenth@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Implement test coverage for BuildConfigs, including merging using
`|` operator, and running `popen_cmake`.
BUG=b:178642582
BRANCH=none
TEST=pytest
Cq-Depend: chromium:2654350
Signed-off-by: Jack Rosenthal <jrosenth@chromium.org>
Change-Id: I07d8e913af999dac4f66f904b9d53e974e57eb40
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2656515
Reviewed-by: Simon Glass <sjg@chromium.org>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This change allows users of the multiproc logging functionality
to override the log level used for a given file descriptor.
This is particularly useful when addressing an issue with ninja
builds that print everything to stdout (including errors).
BRANCH=none
BUG=b:176364631
TEST=introduce a bug then run zmake build $BUILD_DIR, observe
most output is suppressed, but after the error takes place,
all logging is made to CRITICAL.
TEST=add unit test
Signed-off-by: Yuval Peress <peress@chromium.org>
Change-Id: I1dba8a06f0dfebe91e49829dbb8d91f1b71f58f5
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2625550
Reviewed-by: Jack Rosenthal <jrosenth@chromium.org>
Reviewed-by: Simon Glass <sjg@chromium.org>
|
|
This change copies the zmake tests from zephyr-chrome over to
platform/ec to match the tool's relocation.
BRANCH=none
BUG=b:177003034
TEST=pytest zephyr/zmake/tests
TEST=zmake testall
Signed-off-by: Yuval Peress <peress@chromium.org>
Change-Id: I764af51cfcea6821f3bc9274cdfbf42b2da4fb7e
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2622503
|