diff options
| author | Edward O'Callaghan <quasisec@google.com> | 2021-05-17 21:10:23 +1000 |
|---|---|---|
| committer | Edward O'Callaghan <quasisec@chromium.org> | 2021-05-18 12:53:09 +0000 |
| commit | 16c62a791d0a311bb040812b03c33881ab641a8e (patch) | |
| tree | 45e21a39ff76215cc64164f34fb4d2d52073aebc | |
| parent | 11a35980defbf800e33e77cc7a82e89c33865b25 (diff) | |
| download | flashrom-16c62a791d0a311bb040812b03c33881ab641a8e.tar.gz flashrom-16c62a791d0a311bb040812b03c33881ab641a8e.tar.bz2 flashrom-16c62a791d0a311bb040812b03c33881ab641a8e.zip | |
meson.build: Fix cmocka to be optional at configure-time
While building with meson, however without libcmocka
available, attempts are made to fetch a copy to build via
the wrap mechanism. However in hermetic build enviroments
this causes hard failure as the dependency declaration of
cmocka is not optional. Fix this to ensure flashrom can build
hermetically in images without libcmocka available.
BUG=none
BRANCH=none
TEST=`
$ mkdir build && cd build/
$ meson --wrap-mode=nodownload --wrap-mode=nofallback ../
$ ninja test # validate configs, builds and no tests are run.
--
$ sudo apt install libcmocka-dev
$ mkdir build && cd build/
$ meson --wrap-mode=nodownload --wrap-mode=nofallback ../
$ ninja test # validate configs, builds and tests are run.
`
Change-Id: Ib59f4dacc14be9b02334ca59b348c19e22963367
Signed-off-by: Edward O'Callaghan <quasisec@google.com>
Reviewed-on: https://review.coreboot.org/c/flashrom/+/54368
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-by: Anastasia Klimchuk <aklm@chromium.org>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
| -rw-r--r-- | meson.build | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/meson.build b/meson.build index 368820c6..81d63a9d 100644 --- a/meson.build +++ b/meson.build @@ -462,7 +462,8 @@ subdir('util') # unit-test framework cmocka_dep = dependency( 'cmocka', - fallback: ['cmocka', 'cmocka_dep'] + fallback: ['cmocka', 'cmocka_dep'], + required: false ) flashrom_test_dep = declare_dependency( include_directories : include_directories('.'), @@ -481,4 +482,6 @@ flashrom_test_dep = declare_dependency( ], ) -subdir('tests') +if cmocka_dep.found() + subdir('tests') +endif |