Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

Fix subcrate test compile #8862

Merged
merged 2 commits into from
Jun 11, 2018
Merged

Fix subcrate test compile #8862

merged 2 commits into from
Jun 11, 2018

Conversation

sorpaas
Copy link
Collaborator

@sorpaas sorpaas commented Jun 11, 2018

After #8743, several subcrate dev-dependency features are missed. Because Cargo takes union of feature sets, it's not directly visible when testing against the top-level crate. This PR appends all the missing ones.

@sorpaas sorpaas added A0-pleasereview 🤓 Pull request needs code review. M6-rpcapi 📣 RPC API. labels Jun 11, 2018
@sorpaas sorpaas added this to the 1.12 milestone Jun 11, 2018
@debris debris added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Jun 11, 2018
@debris debris merged commit 09ee6e1 into master Jun 11, 2018
@debris debris deleted the sorpaas/fix-subcrate-test-compile branch June 11, 2018 10:26
dvdplm added a commit that referenced this pull request Jun 11, 2018
* master:
  Fix subcrate test compile (#8862)
  network-devp2p: downgrade logging to debug, add target (#8784)
  Clearing up a comment about the prefix for signing (#8828)
  Disable parallel verification and skip verifiying already imported txs. (#8834)
  devp2p: Move UDP socket handling from Discovery to Host. (#8790)
  Fixed AuthorityRound deadlock on shutdown, closes #8088 (#8803)
  Specify critical release flag per network (#8821)
  Fix `deadlock_detection` feature branch compilation (#8824)
  Use system allocator when profiling memory (#8831)
  added from and to to Receipt (#8756)
tavakyan referenced this pull request in C4Coin/c4coin-parity Jun 14, 2018
* Fix test compile for light, node_filter, service in ethcore

* Fix ipfs, local-store, rpc, secret_store, updater subcrate test compile
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A8-looksgood 🦄 Pull request is reviewed well. M6-rpcapi 📣 RPC API.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants