Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Simplified syslog::unix() by using UnixDatagram::connect() (fixes #1) #5

Merged
merged 1 commit into from
Aug 24, 2015

Conversation

tailhook
Copy link
Contributor

@tailhook tailhook commented Aug 7, 2015

The only thing I'm not sure of, is whether next connect() after error works for all platforms. I've tested only on linux.

@tailhook
Copy link
Contributor Author

@Geal did you have a chance to look at this PR? Anything wrong with it?

@Geal
Copy link
Owner

Geal commented Aug 10, 2015

It looks good, just let me find the time to test it. I was stuck with client work these past days, I might be able to look at it on Wednesday (or whenever I get bored of JS and do some Rust for distraction).

@tailhook
Copy link
Contributor Author

@Geal, did you have a chance to test the pull request?

Geal added a commit that referenced this pull request Aug 24, 2015
Simplified `syslog::unix()` by using `UnixDatagram::connect()` (fixes #1)
@Geal Geal merged commit 2dd4245 into Geal:master Aug 24, 2015
@Geal
Copy link
Owner

Geal commented Aug 24, 2015

It's all good, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants