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

Fixed blooms dir creation #8941

Merged
merged 1 commit into from Jun 22, 2018
Merged

Fixed blooms dir creation #8941

merged 1 commit into from Jun 22, 2018

Conversation

svyatonik
Copy link
Collaborator

@debris I've just replaced create_dir with create_dir_all, but could you please check that the blooms path is what you've expected. On my Ubuntu machine it is /home/svyatonik/.local/share/io.parity.ethereum/chains/DevelopmentChain/db/1484bce8c021f2ca/overlayrecent/db/blooms. I haven't reviewed the blooms PR, so not sure - if this is correct.
@Tbaut Sorry - had a yesterday's morning master without this PR merged in :) This PR should fix the issue

@svyatonik svyatonik added A0-pleasereview 🤓 Pull request needs code review. M4-core ⛓ Core client code / Rust. labels Jun 21, 2018
@svyatonik svyatonik requested a review from debris June 21, 2018 09:24
@5chdn 5chdn added this to the 1.12 milestone Jun 21, 2018
Copy link
Collaborator

@niklasad1 niklasad1 left a comment

Choose a reason for hiding this comment

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

LGTM!

@tomusdrw tomusdrw added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Jun 21, 2018
@debris debris merged commit 6be5744 into master Jun 22, 2018
@debris debris deleted the fix_blooms_dir_creation branch June 22, 2018 08:52
dvdplm added a commit that referenced this pull request Jun 22, 2018
* master:
  rpc: cap gas limit of local calls (#8943)
  docs: update changelogs (#8931)
  ethcore: fix compilation when using slow-blocks or evm-debug features (#8936)
  fixed blooms dir creation (#8941)
  Update hardcoded headers (#8925)
ordian added a commit to ordian/parity that referenced this pull request Jun 27, 2018
…rp_sync_on_light_client

* 'master' of https://github.com/paritytech/parity:
  deps: bump fs-swap (openethereum#8953)
  Eliminate some more `transmute()` (openethereum#8879)
  Restrict vault.json permssion to owner and using random suffix for temp vault.json file (openethereum#8932)
  print SS.self_public when starting SS node (openethereum#8949)
  scripts: minor improvements (openethereum#8930)
  rpc: cap gas limit of local calls (openethereum#8943)
  docs: update changelogs (openethereum#8931)
  ethcore: fix compilation when using slow-blocks or evm-debug features (openethereum#8936)
  fixed blooms dir creation (openethereum#8941)
  Update hardcoded headers (openethereum#8925)
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

5 participants