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

OSX port scan and error handling #89

Merged
merged 5 commits into from Aug 19, 2019

Conversation

@nabijaczleweli
Copy link
Collaborator

commented Aug 19, 2019

  1. Properly handle OSX's "Address already in use (os error 48)." error message. Fixes #88
  2. Improved error handling to always include error info

Note: do not merge this with a merge commit. Do an FF merge if you need to but better yet leave this to me and I'll do it cleanly.

Fold down Error::Io into Error. Replace Error::print_error() with a D…
…isplay impl. Remove Error::exit_value()

@nabijaczleweli nabijaczleweli merged commit 61ce23c into master Aug 19, 2019

4 checks passed

continuous-integration/appveyor/branch AppVeyor build succeeded
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.