Skip to content

[20117] Fix setaffinity directive for Android#4125

Merged
EduPonz merged 1 commit intomasterfrom
bugfix/thread_affinity_android
Dec 12, 2023
Merged

[20117] Fix setaffinity directive for Android#4125
EduPonz merged 1 commit intomasterfrom
bugfix/thread_affinity_android

Conversation

@Mario-DL
Copy link
Copy Markdown
Contributor

@Mario-DL Mario-DL commented Dec 12, 2023

Description

This little PR fixes the set_affinity call when compiling for Android since pthread_setaffinity_np() is not exposed.

Contributor Checklist

  • Commit messages follow the project guidelines.
  • The code follows the style guidelines of this project.
  • N/A Tests that thoroughly check the new feature have been added/Regression tests checking the bug and its fix have been added; the added tests pass locally
  • N/A Any new/modified methods have been properly documented using Doxygen.
  • Changes are ABI compatible.
  • Changes are API compatible.
  • N/A New feature has been added to the versions.md file (if applicable).
  • N/A New feature has been documented/Current behavior is correctly described in the documentation.
  • N/A Applicable backports have been included in the description.

Reviewer Checklist

  • The PR has a milestone assigned.
  • Check contributor checklist is correct.
  • Check CI results: changes do not issue any warning.
  • Check CI results: failing tests are unrelated with the changes.

Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>
@Mario-DL Mario-DL added the no-test Skip CI tests if PR marked with this label label Dec 12, 2023
@Mario-DL Mario-DL added this to the v2.13.1 milestone Dec 12, 2023
@MiguelCompany MiguelCompany added the ci-pending PR which CI is running label Dec 12, 2023
@EduPonz EduPonz modified the milestones: v2.13.1, v2.13.0 Dec 12, 2023
@Mario-DL
Copy link
Copy Markdown
Contributor Author

Sorry, I mistakenly aborted windows v142, but I think we could go with the Github Action one

@EduPonz EduPonz merged commit 3aa012d into master Dec 12, 2023
@EduPonz EduPonz deleted the bugfix/thread_affinity_android branch December 12, 2023 15:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-pending PR which CI is running no-test Skip CI tests if PR marked with this label

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants