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

OpenSUSE (and presumably SEL) needs <errno.h> included in ib_verbs.h. #30

Closed
wants to merge 2 commits into from
Closed

Conversation

tonycurtis
Copy link
Contributor

RH and derivatives appear to have pulled in errno somewhere else. This fix has no effect if so.

derivatives appear to have pulled in errno somewhere else.
@mellanox-github
Copy link
Contributor

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
http://bgate.mellanox.com:8000/jenkins-secure/job/gh-ucx-pr/54/
Test PASSed.

@shamisp
Copy link
Contributor

shamisp commented Nov 25, 2014

👍

@shamisp
Copy link
Contributor

shamisp commented Nov 25, 2014

Actually I missed one point. All the includes have to go after "config" can you please move it few lines down. Thanks

@shamisp
Copy link
Contributor

shamisp commented Nov 25, 2014

Also, please use git commit --amend, so it will append to the existing commit. Also it will require "-f" for the push.

@tonycurtis tonycurtis closed this Nov 25, 2014
@shamisp
Copy link
Contributor

shamisp commented Nov 25, 2014

why did you close ?

@tonycurtis
Copy link
Contributor Author

On Nov 24, 2014, at 9:50 PM, Pasha <notifications@github.mirror.nvdadr.commailto:notifications@github.com> wrote:

why did you close ?

I don’t know. trying to work out what to do…

@shamisp
Copy link
Contributor

shamisp commented Nov 25, 2014

Do you have re-open button somewhere ? They used to have it.

@tonycurtis
Copy link
Contributor Author

On Nov 24, 2014, at 9:55 PM, Pasha <notifications@github.mirror.nvdadr.commailto:notifications@github.com> wrote:

Do you have re-open button somewhere ? They used to have it.

No. I am confused. “Create another pull request”?

@shamisp
Copy link
Contributor

shamisp commented Nov 25, 2014

Ok, please create another.

@yosefe yosefe deleted the errno-fix branch November 26, 2014 08:38
amaslenn pushed a commit to amaslenn/ucx that referenced this pull request Jun 2, 2020
leibin2014 referenced this pull request in leibin2014/ucx Sep 10, 2020
UCT/RC/MLX5: Dispatch arbiter when CQ resources released in error flow
shizhibao pushed a commit to shizhibao/ucx that referenced this pull request Jan 16, 2021
Fix UT test for discontig datatype
dmitrygx pushed a commit to dmitrygx/ucx that referenced this pull request Dec 1, 2021
evgeny-leksikov pushed a commit to evgeny-leksikov/ucx that referenced this pull request Apr 19, 2022
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.

3 participants