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

Windows fixes #6921

Merged
merged 5 commits into from
Nov 1, 2017
Merged

Windows fixes #6921

merged 5 commits into from
Nov 1, 2017

Conversation

arkpar
Copy link
Collaborator

@arkpar arkpar commented Oct 29, 2017

No description provided.

@arkpar arkpar added A0-pleasereview 🤓 Pull request needs code review. M4-core ⛓ Core client code / Rust. labels Oct 29, 2017
rust-toolchain Outdated
@@ -1 +0,0 @@
stable
Copy link
Collaborator

Choose a reason for hiding this comment

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

why did you remove this?

parity/rpc.rs Outdated
if let Some(dir) = path.parent() {
::std::fs::create_dir_all(&dir)
.map_err(|err| format!("Unable to create IPC directory at {}: {}", dir.display(), err))?;
if !cfg!(windows) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

why not on windows? Please comment it in the code

@5chdn 5chdn added this to the Patch milestone Oct 30, 2017
@svyatonik svyatonik added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Oct 31, 2017
@kirushik kirushik merged commit 4b3c26f into master Nov 1, 2017
@debris debris deleted the windows-fixes branch November 2, 2017 19:20
@arkpar arkpar added the B0-patch label Nov 8, 2017
debris pushed a commit that referenced this pull request Nov 13, 2017
@debris debris mentioned this pull request Nov 13, 2017
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.

5 participants