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

Windows named pipes #1254

Merged
merged 5 commits into from
Jun 11, 2016
Merged

Windows named pipes #1254

merged 5 commits into from
Jun 11, 2016

Conversation

NikVolf
Copy link
Contributor

@NikVolf NikVolf commented Jun 10, 2016

No description provided.

@NikVolf NikVolf added the A3-inprogress ⏳ Pull request is in progress. No review needed at this stage. label Jun 10, 2016
@NikVolf NikVolf changed the title Windows pipes Windows named pipes Jun 10, 2016
@NikVolf NikVolf added A0-pleasereview 🤓 Pull request needs code review. and removed A3-inprogress ⏳ Pull request is in progress. No review needed at this stage. labels Jun 10, 2016
@@ -238,6 +238,7 @@ fn execute_client(conf: Configuration, spec: Spec, client_config: ClientConfig)

// setup ipc rpc
let _ipc_server = rpc::new_ipc(conf.ipc_settings(), &dependencies);
println!("IPC: {}", conf.ipc_settings());
Copy link
Collaborator

@arkpar arkpar Jun 11, 2016

Choose a reason for hiding this comment

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

Should be removed or logged with debug!

@arkpar arkpar merged commit f6582e7 into windows Jun 11, 2016
@debris debris deleted the windows-pipes branch June 13, 2016 10:03
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-pleasereview 🤓 Pull request needs code review.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants