Paths

Table of Contentst

Differential D7398

[CI] Split the secp256k1 build in parts
ClosedPublic
Actions

Authored by Fabien on Sep 8 2020, 16:10.

Details

Reviewers

jasonbcox

Group Reviewers

Restricted Project

Commits

rABC10978b1799c9: [CI] Split the secp256k1 build in parts

Summary

The build actually runs 3 builds serially, this diffs make them
different configurations so they can run parallel.

Test Plan

./contrib/teamcity/build-configurations.py build-secp256k1
./contrib/teamcity/build-configurations.py build-secp256k1-endomorphism
./contrib/teamcity/build-configurations.py build-secp256k1-java

Diff Detail

Repository

rABC Bitcoin ABC

Branch

ci_split_secp256k1

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 12668
Build 25437: Build Diff	build-diff · build-clang-tidy · build-clang-10 · build-without-wallet
Build 25436: arc lint + arc unit

Event Timeline

Fabien created this revision.Sep 8 2020, 16:10

Herald added a reviewer: Restricted Project. · View Herald TranscriptSep 8 2020, 16:10

Fabien requested review of this revision.Sep 8 2020, 16:10

Harbormaster completed remote builds in B12668: Diff 23379.Sep 8 2020, 16:32

Fabien added a child revision: D7403: [CI] Allow to configure a post build script.Sep 8 2020, 20:13

jasonbcox requested changes to this revision.Sep 8 2020, 20:34

jasonbcox added a subscriber: jasonbcox.

jasonbcox added inline comments.

contrib/teamcity/build-configurations.yml
272	This is not running `ninja check-secp256k1-java` like the script used to.

This revision now requires changes to proceed.Sep 8 2020, 20:34

Fabien requested review of this revision.Sep 8 2020, 20:37

Fabien added inline comments.

contrib/teamcity/build-configurations.yml
272	`check-secp256k1` is a superset of `check-secp256k1-java`, so it has an increased scope but still running the java tests

jasonbcox accepted this revision.Sep 8 2020, 20:40

This revision is now accepted and ready to land.Sep 8 2020, 20:40

Closed by commit rABC10978b1799c9: [CI] Split the secp256k1 build in parts (authored by Fabien). · Explain WhySep 9 2020, 06:36

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABC10978b1799c9: [CI] Split the secp256k1 build in parts.

Fabien removed a child revision: D7403: [CI] Allow to configure a post build script.Sep 9 2020, 10:30

Revision Contents
Changeset List

Path

Size

contrib/

teamcity/

build-configurations.yml

33 lines

builds/

build-secp256k1.sh

Diff 23379

View Options

contrib/teamcity/build-configurations.yml