Conversation
.circleci/config.yml
Outdated
ekiden-key-manager | ||
# Build the contract. | ||
- run: cargo ekiden build-contract --output-identity | ||
# Run the contract tests. | ||
- run: cargo test | ||
# Build the client. | ||
- run: | ||
command: cargo build | ||
command: CARGO_BUILD_TARGET_DIR=../target cargo build |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe move this to the client Cargo.toml file.
maybe just make this whole repo a workspace, even
8479509
to
9b37a21
Compare
6235a98
to
5bf9438
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This switches the ekiden dependencies from a tag to master. What is the best way to maintain separate configurations for the two contract-evm testnets we discussed?
yup that's temporary |
broken on alpha.4 incompatibility |
Once oasisprotocol/oasis-core#172 is merged, we can tag alpha.5 and revisit. |
# Build the contract. | ||
- run: cargo ekiden build-contract --output-identity | ||
# Run the contract tests. | ||
- run: cargo test | ||
# Build the client. | ||
- run: | ||
command: cargo build | ||
environment: | ||
CARGO_BUILD_TARGET_DIR: ../target |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
rust-lang/cargo#1657
CARGO_TARGET_DIR
works too, and I think that one is better documented
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me
.circleci/config.yml
Outdated
@@ -87,7 +106,8 @@ workflows: | |||
- deploy: | |||
filters: | |||
branches: | |||
only: master | |||
# %%% change back to master |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
reminder to change back to master
Configure the build commands in CI to use a shared target dir. Also install programs in debug mode to further share compiled libraries. Reduces build time from ~15 min to ~8 min. Deploy time goes from ~13 min to ~10 min.
fixes #2
depends: oasisprotocol/oasis-core#168
tasks before merge:
tagged versions (can be done already with alpha.4)master branch