ci: Check for `git` before its usage #27718

pull hebasto wants to merge 1 commits into bitcoin:master from hebasto:230522-ci-git changing 1 files +1 −1

hebasto commented at 1:35 PM on May 22, 2023: member
Fixes https://cirrus-ci.com/task/6690296436621312.

https://api.cirrus-ci.com/v1/task/6690296436621312/logs/build.log:
```
./ci/test/01_base_install.sh: line 11: git: command not found
```
DrahtBot commented at 1:35 PM on May 22, 2023: contributor



The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.



Reviews

See the guideline for information on the review process. A summary of reviews will appear here.
DrahtBot added the label Tests on May 22, 2023
maflcko commented at 1:38 PM on May 22, 2023: member

The warning is harmless. If git isn't installed, we can be sure that base-install-done is not true
hebasto commented at 1:44 PM on May 22, 2023: member

The warning is harmless. If git isn't installed, we can be sure that base-install-done is not true

You mean, checking for git binary, command -v git?
ci: Check for `git` before its usage 69769d0b40
hebasto force-pushed on May 22, 2023
hebasto commented at 1:51 PM on May 22, 2023: member

The warning is harmless. If git isn't installed, we can be sure that base-install-done is not true

Reworked.
hebasto renamed this:
~~ci: Ensure git is installed before usage~~
ci: Check for `git` before its usage
on May 22, 2023
hebasto closed this on May 22, 2023
bitcoin locked this on May 21, 2024

Contributors

Labels