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

Fix unix-like platforms build #722

Merged
merged 1 commit into from
May 9, 2024
Merged

Conversation

sthibaul
Copy link
Contributor

@sthibaul sthibaul commented May 9, 2024

semaphore.h is used in various places, and the implementation based on unix semaphores is largely supported by unix platforms.

Fixes:
src/internal_modules/roc_ctl/control_task.h:20:10: fatal error: roc_core/semaphore.h: No such file or directory
20 | #include "roc_core/semaphore.h"
| ^~~~~~~~~~~~~~~~~~~~~~

semaphore.h is used in various places, and the implementation based on
unix semaphores is largely supported by unix platforms.

Fixes:
src/internal_modules/roc_ctl/control_task.h:20:10: fatal error: roc_core/semaphore.h: No such file or directory
   20 | #include "roc_core/semaphore.h"
      |          ^~~~~~~~~~~~~~~~~~~~~~
Copy link

github-actions bot commented May 9, 2024

🤖 Upon creation, pull request description does not have a link to an issue. If there is a related issue, please add it to the description using any of the supported formats.

@gavv
Copy link
Member

gavv commented May 9, 2024

Thanks. I forgot that we have semaphore only for posix_ext and darwin right now.

@gavv gavv merged commit 395b112 into roc-streaming:develop May 9, 2024
35 checks passed
@gavv gavv added the contribution A pull-request by someone else except maintainers label May 9, 2024
@gavv gavv added this to the next milestone May 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contribution A pull-request by someone else except maintainers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants