diff options
| author |   Martin Jansa <Martin.Jansa@gmail.com> | 2021-03-11 12:01:44 +0100 |
|---|---|---|
| committer |   Khem Raj <raj.khem@gmail.com> | 2021-03-11 10:06:02 -0800 |
| commit | af7851aa40a02e2b3bccc423d98759bb24553511 (patch) | |
| tree | fec0e224ecb3fbcd2ad08d5aa6535c26b95c88be /meta-oe/recipes-support/ceres-solver | |
| parent | 9be6b4f5a2ec857475626c74457a94b8d9236fd5 (diff) | |
| download | meta-openembedded-af7851aa40a02e2b3bccc423d98759bb24553511.tar.gz | |
ceres-solver: prevent fetching git hook during do_configure
* today I've found 2 jenkins jobs stuck way too long sitting in this do_configure
Bitbake still alive (5000s)
Bitbake still alive (10000s)
Bitbake still alive (15000s)
Bitbake still alive (20000s)
Bitbake still alive (25000s)
Bitbake still alive (30000s)
... manually killed, the CMake ...
ERROR: ceres-solver-1.14.0-r0 do_configure: Execution of 'ceres-solver/1.14.0-r0/temp/run.do_configure.39438' failed with exit code 143:
...
| -- Detected Ceres being used as a git submodule, adding commit hook for Gerrit to: ceres-solver/1.14.0-r0/git/.git
| ceres-solver/1.14.0-r0/temp/run.do_configure.39438: line 213: 39485 Terminated cmake -G 'Ninja' -DCMAKE_MAKE_PROGRAM=ninja ...
I've seen it with dunfell and gatesgarth, but master has the same
ADD_GERRIT_COMMIT_HOOK function (just in newer ceres-solver release),
so probably needs the same.
Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
Signed-off-by: Khem Raj <raj.khem@gmail.com>
Diffstat (limited to 'meta-oe/recipes-support/ceres-solver')
| -rw-r--r-- | meta-oe/recipes-support/ceres-solver/ceres-solver_2.0.0.bb | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/meta-oe/recipes-support/ceres-solver/ceres-solver_2.0.0.bb b/meta-oe/recipes-support/ceres-solver/ceres-solver_2.0.0.bb index 28d0868c48..7feba4a5a2 100644 --- a/meta-oe/recipes-support/ceres-solver/ceres-solver_2.0.0.bb +++ b/meta-oe/recipes-support/ceres-solver/ceres-solver_2.0.0.bb @@ -13,6 +13,14 @@ S = "${WORKDIR}/git" inherit cmake +do_configure_prepend() { + # otherwise https://github.com/ceres-solver/ceres-solver/blob/0b748597889f460764f6c980a00c6f502caa3875/cmake/AddGerritCommitHook.cmake#L68 + # will try to fetch https://ceres-solver-review.googlesource.com/tools/hooks/commit-msg durind do_configure + # which sometimes gets stuck (as there is no TIMEOUT set in DOWNLOAD) + # and we really don't need Gerrit's Change-Id tags when just building this + touch ${S}/.git/hooks/commit-msg +} + # We don't want path to eigen3 in ceres-solver RSS to be # used by components which use CeresConfig.cmake from their # own RSS |