Skip to content

Add include to fix std::counting_semaphore usage#1333

Merged
jcoupey merged 1 commit intoVROOM-Project:masterfrom
cho-m:fix/include-semaphore
Mar 17, 2026
Merged

Add include to fix std::counting_semaphore usage#1333
jcoupey merged 1 commit intoVROOM-Project:masterfrom
cho-m:fix/include-semaphore

Conversation

@cho-m
Copy link
Copy Markdown
Contributor

@cho-m cho-m commented Mar 13, 2026

Issue

Build failure seen on macOS 26 when building with Clang:

In file included from problems/vrp.cpp:10:
./problems/vrp.h:207:10: error: no member named 'counting_semaphore' in namespace 'std'
  207 |     std::counting_semaphore<32> semaphore(actual_nb_threads);
      |     ~~~~~^
./problems/vrp.h:207:33: error: use of undeclared identifier 'semaphore'
  207 |     std::counting_semaphore<32> semaphore(actual_nb_threads);
      |                                 ^

Seemed like simple change so directly opened pull request, but can create & reference a separate issue if preferred.

Tasks

  • review

@jcoupey jcoupey added this to the v1.16.0 milestone Mar 16, 2026
@jcoupey jcoupey merged commit 3bd437a into VROOM-Project:master Mar 17, 2026
4 checks passed
@cho-m cho-m deleted the fix/include-semaphore branch March 18, 2026 00:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants