Details

Reviewers

deadalnix
mbhinder
freetrader

Group Reviewers

Restricted Project

Commits

rSTAGING675f0dca640a: Initial check-in of teamcity builds
rABC675f0dca640a: Initial check-in of teamcity builds

Summary

This includes the Kotlin scripts for teamcity builds as they
are currently setup on https://build.bitcoinabc.org

Test Plan

Merge, and set teamcity to use the configuration.

Diff Detail

Repository

rABC Bitcoin ABC

Branch

teamcity

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 1221
Build 1221: arc lint + arc unit

Event Timeline

schancel created this revision.Dec 2 2017, 17:35

Herald added a reviewer: Restricted Project. · View Herald TranscriptDec 2 2017, 17:35

Remove extraneous file

To be honest, I lack a bit the knowledge to review this properly. I would recommend using ABC in capital if that's possible as this is how it is used elsewhere.

Maybe @mbhinder could do a preliminary pass ?

LGTM

.teamcity/BitcoinAbc/vcsRoots/Test_SshVcsReviewsBitcoinabcOrg2221sourceBitcoinAbcGit.kt
6 ↗	(On Diff #1899)	nit: Better naming?

It's all autogenerated. I'll be putting up another patch that's more complete in a bit, but I'm going to leave it all default and we can work on it as we need to change it..

I need to investigate this teamcity functionality more. For now, we can run a checked in shell script.

Initial teamcity builds

Newline at EOF

Harbormaster completed remote builds in B1369: Diff 2118.Dec 13 2017, 23:25

deadalnix accepted this revision.Dec 13 2017, 23:25

This revision is now accepted and ready to land.Dec 13 2017, 23:25

teamcity mentioned this in T131: Revision D744 broke builds.Dec 14 2017, 02:05

teamcity added a subscriber: teamcity.

This comment was removed by schancel.

teamcity mentioned this in T132: Revision D744 broke builds.Dec 14 2017, 02:06

This comment was removed by schancel.

teamcity mentioned this in T133: Revision D744 broke builds.Dec 14 2017, 02:06