ci: Merge C++17 build with one of the existing ones #18829

pull MarcoFalke wants to merge 1 commits into bitcoin:master from MarcoFalke:2004-ciSpeedupCxx17 changing 3 files +2 −20

MarcoFalke commented at 1:54 PM on April 30, 2020: member

No need to spin up an extra vm for each pull request for a simple sanity check that any of the other already running machines can test.
MarcoFalke force-pushed on Apr 30, 2020
DrahtBot added the label Tests on Apr 30, 2020
laanwj commented at 2:55 PM on April 30, 2020: member

Continues #18591 (ping @sipa).
MarcoFalke commented at 3:22 PM on April 30, 2020: member

Background is that the travis queue has a huge backlog. So we could either throw some more money at them, or (what seems simpler) not have that many configurations, most of which can be merged without any downside.
ci: Merge C++17 build with one of the existing ones aaaacff107

in .travis.yml:152 in fa37947fc3 outdated

 147 | @@ -148,11 +148,6 @@ jobs:
 148 |        env: >-
 149 |          FILE_ENV="./ci/test/00_setup_env_native_valgrind.sh"
 150 |  
 151 | -    - stage: test
 152 | -      name: 'x86_64 Linux  [GOAL: install]  [bionic]  [no depends, only system libs, c++17]'

sipa commented at 4:28 PM on April 30, 2020:

Maybe add a c++17 marker to the qt5 job it's merged with?

MarcoFalke commented at 4:51 PM on April 30, 2020:

Thanks, done

MarcoFalke force-pushed on Apr 30, 2020
sipa commented at 4:57 PM on April 30, 2020: member

ACK
MarcoFalke commented at 6:04 PM on April 30, 2020: member

Still picked up: https://travis-ci.org/github/bitcoin/bitcoin/jobs/681586851#L3194
MarcoFalke merged this on Apr 30, 2020
MarcoFalke closed this on Apr 30, 2020
MarcoFalke deleted the branch on May 6, 2020
MarkLTZ referenced this in commit 9dc5b18a30 on Feb 16, 2021
DrahtBot locked this on Aug 16, 2022

Contributors

Labels