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

parity-version pr reopen #7136

Merged
merged 6 commits into from Dec 22, 2017
Merged

parity-version pr reopen #7136

merged 6 commits into from Dec 22, 2017

Conversation

debris
Copy link
Collaborator

@debris debris commented Nov 24, 2017

No description provided.

@debris debris added A0-pleasereview 🤓 Pull request needs code review. M4-core ⛓ Core client code / Rust. labels Nov 24, 2017
removed unused util deps and features

trigger buildbot again

only kvdb links rocksdb

snappy linker issues
@5chdn 5chdn added this to the 1.9 milestone Nov 24, 2017
@5chdn
Copy link
Contributor

5chdn commented Nov 28, 2017

tests fail

@debris
Copy link
Collaborator Author

debris commented Nov 28, 2017

yes, cause linking C++ libraries is somehow broken and behaves in unpredictable way 😕

@arkpar
Copy link
Collaborator

arkpar commented Dec 4, 2017

Needs a rebase. Snappy issues should be fixed in master now.

@5chdn 5chdn added A3-inprogress ⏳ Pull request is in progress. No review needed at this stage. and removed A0-pleasereview 🤓 Pull request needs code review. labels Dec 6, 2017
@debris debris added A0-pleasereview 🤓 Pull request needs code review. and removed A3-inprogress ⏳ Pull request is in progress. No review needed at this stage. labels Dec 19, 2017
Cargo.toml Outdated
@@ -94,13 +95,12 @@ ui-precompiled = [
ui-enabled = ["dapps"]
dapps = ["parity-dapps"]
jit = ["ethcore/jit"]
dev = ["clippy", "ethcore/dev", "ethcore-util/dev", "ethsync/dev", "parity-rpc/dev", "parity-dapps/dev"]
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why is ethcore-util/dev removed from here?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

ethcore-util/dev does not exist any more

Copy link
Collaborator

Choose a reason for hiding this comment

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

Fair enough, anyway see #7355

@tomusdrw tomusdrw added A7-looksgoodtestsfail 🤖 Pull request is reviewed well, but cannot be merged due to tests failing. and removed A0-pleasereview 🤓 Pull request needs code review. labels Dec 22, 2017
@tomusdrw tomusdrw added A8-looksgood 🦄 Pull request is reviewed well. and removed A7-looksgoodtestsfail 🤖 Pull request is reviewed well, but cannot be merged due to tests failing. labels Dec 22, 2017
@debris debris merged commit d80dd81 into master Dec 22, 2017
@debris debris deleted the parity-version branch December 22, 2017 13:37
tomusdrw pushed a commit that referenced this pull request Feb 14, 2018
* parity-version module split from util

removed unused util deps and features

trigger buildbot again

only kvdb links rocksdb

snappy linker issues

* rm snappy

* fixed old version imports
5chdn pushed a commit that referenced this pull request Feb 14, 2018
* update back-references more aggressively after answering from cache (#7578)

* Flush keyfiles. Resolves #7632 (#7868)

* Fix wallet import (#7873)

* rpc: generate new account id for imported wallets

* ethstore: handle duplicate wallet filenames

* ethstore: simplify deduplication of wallet file names

* ethstore: do not dedup wallet filenames on update

* ethstore: fix minor grumbles

* parity-version pr reopen (#7136)

* parity-version module split from util

removed unused util deps and features

trigger buildbot again

only kvdb links rocksdb

snappy linker issues

* rm snappy

* fixed old version imports

* Move updater metadata to Cargo.toml of parity-version. (#7832)

* Update version.

* Bump parity version.

* Fix version.

* Fix compilation.
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. M4-core ⛓ Core client code / Rust.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants