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

configure: Restore libuwnind support #401

Merged
merged 1 commit into from
Sep 13, 2024
Merged

Conversation

knard-intel
Copy link

@knard-intel knard-intel commented Sep 9, 2024

Pull Request Description

The commit 3196bbf removed the support of libunwind.
From the message 3196bbf#r143955103, it seems to not be intentional.

DAOS-15596 ticket, Update Argobots to 1.2

Checklist

  • Reference appropriate issues (with "Fixes" or "See" as appropriate)
  • Commits are self-contained and do not do two things at once
  • Commit message is of the form: module: short description and follows good practice
  • Passes whitespace checkers

The commit 3196bbf removed the support
of libunwind.

DAOS-15596 ticket, Update Argobots to 1.2

Signed-off-by: Cedric Koch-Hofer <[email protected]>
@knard-intel
Copy link
Author

@yfguo , could it be possible to add you and @shintaro-iwasaki as reviewers of this PR.
Thanks in advance.

Copy link
Collaborator

@shintaro-iwasaki shintaro-iwasaki left a comment

Choose a reason for hiding this comment

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

Thanks, @knard-intel! It seems that I removed it by mistake. It looks good to me!
Please ask @yfguo to merge this PR.

@knard-intel
Copy link
Author

Thanks, @knard-intel! It seems that I removed it by mistake. It looks good to me! Please ask @yfguo to merge this PR.

@yfguo , could you please land this PR.
Thanks in advance.
Cedric.

@knard-intel
Copy link
Author

@yfguo , please could you tell me if something is missing to land this PR.
I have some DAOS PRs depending on this argobot PR and it could be nice if we could land it.
Thanks in advance.

@yfguo yfguo self-requested a review September 13, 2024 14:32
@yfguo yfguo merged commit bb6faac into pmodels:main Sep 13, 2024
2 checks passed
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.

5 participants