Skip to content

provide an alternative path for robustlock files under QNX #1776

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

Merged
merged 4 commits into from
Feb 23, 2021
Merged

provide an alternative path for robustlock files under QNX #1776

merged 4 commits into from
Feb 23, 2021

Conversation

asobhy-qnx
Copy link
Contributor

@asobhy-qnx asobhy-qnx commented Feb 23, 2021

fixes #1766 in 2.1.x
The robust lock path is changed from /dev/shmem to /var/lock as discussed with @IkerLuengo
First we check if the directory /var/lock exists then mkdir it if it does not then proceed with the normal procedure of creating the locked file.

if PR is approved please cherry pick to 2.0.x as well, thank you.

@richiprosima
Copy link
Contributor

Build status:

  • Linux Build Status
  • Linux aarch64Build Status
  • Mac Build Status
  • Windows Build Status

@MiguelCompany MiguelCompany added the skip-ci Automatically pass CI label Feb 23, 2021
MiguelCompany
MiguelCompany previously approved these changes Feb 23, 2021
@MiguelCompany MiguelCompany merged commit f1cd8aa into eProsima:2.1.x Feb 23, 2021
MiguelCompany added a commit that referenced this pull request Feb 23, 2021
* provide an alternative path for robustlock files under QNX

Signed-off-by: Ahmed Sobhy <[email protected]>

* Uncrustify

* Uncrustify

* Uncrustify

Co-authored-by: Miguel Company <[email protected]>
@asobhy-qnx asobhy-qnx deleted the qnx-support branch February 23, 2021 15:11
@asobhy-qnx
Copy link
Contributor Author

can you cherry pick to 2.0.1 as well?

@MiguelCompany
Copy link
Member

can you cherry pick to 2.0.1 as well?

Commit 81cda6a on 2.0.x is a cherry pick of this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip-ci Automatically pass CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants