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

Add daemonization. #459

Merged
merged 8 commits into from Feb 18, 2016
Merged

Add daemonization. #459

merged 8 commits into from Feb 18, 2016

Conversation

gavofyork
Copy link
Contributor

No description provided.

@gavofyork gavofyork added the A0-pleasereview 🤓 Pull request needs code review. label Feb 18, 2016
}
if self.args.cmd_daemon {
let daemonize = Daemonize::new()
.pid_file("/tmp/parity.pid") // Every method except `new` and `start`
Copy link
Collaborator

Choose a reason for hiding this comment

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

I guess it won't work on Windows? :)

parity [options] [ --no-bootstrap | <enode>... ]

Options:
--chain CHAIN Specify the blockchain type. CHAIN may be either a JSON chain specification file
or frontier, mainnet, morden, or testnet [default: frontier].
or frontier, mainnet, morden, or testnet [default: frontier].
Copy link
Collaborator

Choose a reason for hiding this comment

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

misaligned?

@gavofyork
Copy link
Contributor Author

yeah - still needs work.

@gavofyork gavofyork added A3-inprogress ⏳ Pull request is in progress. No review needed at this stage. and removed A0-pleasereview 🤓 Pull request needs code review. labels Feb 18, 2016
@gavofyork gavofyork added A0-pleasereview 🤓 Pull request needs code review. and removed A3-inprogress ⏳ Pull request is in progress. No review needed at this stage. labels Feb 18, 2016
@gavofyork gavofyork mentioned this pull request Feb 18, 2016
@arkpar arkpar added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Feb 18, 2016
@gavofyork
Copy link
Contributor Author

merging since this compiles locally and build failure is unrelated.

gavofyork pushed a commit that referenced this pull request Feb 18, 2016
@gavofyork gavofyork merged commit f84ffdb into master Feb 18, 2016
@debris debris deleted the daemonize branch March 10, 2016 09:57
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants