Skip to content
This repository was archived by the owner on Jul 27, 2022. It is now read-only.

Problem: (CRO-562) tiny-keccak 2.0 fails to compile#576

Merged
bors[bot] merged 1 commit intocrypto-com:masterfrom
devashishdxt:keccak-2.0
Nov 12, 2019
Merged

Problem: (CRO-562) tiny-keccak 2.0 fails to compile#576
bors[bot] merged 1 commit intocrypto-com:masterfrom
devashishdxt:keccak-2.0

Conversation

@devashishdxt
Copy link
Copy Markdown
Contributor

Solution: Made necessary changes for tiny-keccak = 2.0

Solution: Made necessary changes for `tiny-keccak = 2.0`
@devashishdxt devashishdxt changed the title Problem: tiny-keccak 2.0 fails to compile Problem: (CRO-562) tiny-keccak 2.0 fails to compile Nov 12, 2019
@tomtau
Copy link
Copy Markdown
Contributor

tomtau commented Nov 12, 2019

bors try

bors bot added a commit that referenced this pull request Nov 12, 2019
@tomtau
Copy link
Copy Markdown
Contributor

tomtau commented Nov 12, 2019

bors cancel

@tomtau tomtau self-requested a review November 12, 2019 04:29
Copy link
Copy Markdown
Contributor

@tomtau tomtau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

bors r+

@codecov
Copy link
Copy Markdown

codecov bot commented Nov 12, 2019

Codecov Report

Merging #576 into master will decrease coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #576      +/-   ##
==========================================
- Coverage   67.49%   67.49%   -0.01%     
==========================================
  Files         124      124              
  Lines       14363    14365       +2     
==========================================
+ Hits         9695     9696       +1     
- Misses       4668     4669       +1
Impacted Files Coverage Δ
chain-core/src/init/address.rs 73.82% <100%> (+0.35%) ⬆️
chain-core/src/tx/fee/mod.rs 87.68% <0%> (-0.5%) ⬇️

bors bot added a commit that referenced this pull request Nov 12, 2019
535: Problem: (CRO-392) outdated dependencies in client's storage encryption r=tomtau a=linfeng-crypto

Solution:
- use crate `aes-gcm-siv` and `aead` instead of `miscreant`
- use crate `rust-argon2` to the `passphrase` to a constant length,  and store the `salt` at the end of the encrypted data.



576: Problem: (CRO-562) tiny-keccak 2.0 fails to compile r=tomtau a=devashishdxt

Solution: Made necessary changes for `tiny-keccak = 2.0`

Co-authored-by: ylf <cxwcylf@126.com>
Co-authored-by: Devashish Dixit <devashish@crypto.com>
@bors
Copy link
Copy Markdown
Contributor

bors bot commented Nov 12, 2019

@bors
Copy link
Copy Markdown
Contributor

bors bot commented Nov 12, 2019

Build failed (retrying...)

bors bot added a commit that referenced this pull request Nov 12, 2019
576: Problem: (CRO-562) tiny-keccak 2.0 fails to compile r=tomtau a=devashishdxt

Solution: Made necessary changes for `tiny-keccak = 2.0`

Co-authored-by: Devashish Dixit <devashish@crypto.com>
@bors
Copy link
Copy Markdown
Contributor

bors bot commented Nov 12, 2019

@bors bors bot merged commit 9929580 into crypto-com:master Nov 12, 2019
@devashishdxt devashishdxt deleted the keccak-2.0 branch November 12, 2019 08:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants