Closed hebasto closed 10 months ago
@MarcoFalke
After the last push, the CI merge_base_script
works in a wrong way:
bash -c "$PACKAGE_MANAGER_INSTALL git"
Hit:1 https://mirror.hetzner.com/ubuntu/packages lunar InRelease
Hit:2 https://mirror.hetzner.com/ubuntu/packages lunar-updates InRelease
Hit:3 https://mirror.hetzner.com/ubuntu/packages lunar-backports InRelease
Hit:4 https://mirror.hetzner.com/ubuntu/security lunar-security InRelease
Reading package lists...
Reading package lists...
Building dependency tree...
Reading state information...
git is already the newest version (1:2.39.2-1ubuntu1.1).
0 upgraded, 0 newly installed, 0 to remove and 6 not upgraded.
if [ "$CIRRUS_PR" = "" ]; then exit 0; fi
git fetch --depth=1 $CIRRUS_REPO_CLONE_URL "pull/${CIRRUS_PR}/merge"
From https://github.com/bitcoin-core/gui-qml
* branch refs/pull/359/merge -> FETCH_HEAD
git checkout FETCH_HEAD
Previous HEAD position was 592da16 Adjust code according to changes in the main repository
HEAD is now at 6f6e8b4 Merge b1f441ff3c280b4a59ccd2c15658628877830068 into 67895f77f2da6471c5c3b67988fde7f932f77782
M ci/test/00_setup_env_native_asan.sh
The HEAD
must be at Merge 592da1615079d98afd81c074f0447122b718b125 into 67895f77f2da6471c5c3b67988fde7f932f77782
, no?
UPD. Or is it just an intermittent GitHub issue?
UPD. Or is it just an intermittent GitHub issue?
~Looks like that. Re-run fixes it.~
UPD. It still happens.
Going to merge this PR to unlock migration from non-free Cirrus CI to free GitHub Actions.
Hoping someone will make post-merge review soon.
utACK 14ddf6169358526ee5f349da2a7102dcd5dcd232.
ACK merge commit b7cdd31be
Light code review:
modernize-use-default-member-init
clang-tidy warnings
Sync with the main repo up to the latest https://github.com/bitcoin/bitcoin/commit/ab42b2ebdbf61225e636e4c00068fd29b2790d41, which includes the recent changes in the CI.
There is no downloadable artifacts support for now. It will be done in a separated PR(s).
Additionally:
modernize-use-default-member-init
clang-tidy warnings.Guix builds: