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.

Sign up for GitHub

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

Fix/local bind #5819

Merged
merged 2 commits into from
Oct 2, 2018
Merged

Fix/local bind #5819

merged 2 commits into from
Oct 2, 2018

Commits on Oct 2, 2018

  1. Provide the correct socklen to bind. 

    Get Brian's patch from open-mpi#5825 and his log message:
Fix a failure in binding the initiating side of a connection
on MacOS. MacOS doesn't like passing the size of the storage
structure (sockaddr_storage) instead of the expected size of
the structure (sockaddr_in or sockaddr_in6), which was causing
bind() failures. This patch simply changes the structure size
to the expected size.

Add a more clear error message in debug mode.

Signed-off-by: George Bosilca <[email protected]>
    @bosilca
    bosilca committed Oct 2, 2018
    Configuration menu
    Copy the full SHA
    9164e26 View commit details
    Browse the repository at this point in the history

  2. Small pedantic fixes. 

    Signed-off-by: George Bosilca <[email protected]>
    @bosilca
    bosilca committed Oct 2, 2018
    Configuration menu
    Copy the full SHA
    a3a492b View commit details
    Browse the repository at this point in the history