Skip to content

Comments

FreeBSD build#154

Merged
sgerbino merged 2 commits intocppalliance:developfrom
sgerbino:pr/freebsd
Feb 19, 2026
Merged

FreeBSD build#154
sgerbino merged 2 commits intocppalliance:developfrom
sgerbino:pr/freebsd

Conversation

@sgerbino
Copy link
Collaborator

Link pthreads via Threads::Threads in CMakeLists.txt — required on FreeBSD where pthread isn't implicitly linked.
Add missing #include <sys/socket.h> in endpoint_convert.hpp — FreeBSD requires it for sockaddr definitions (Linux provides it transitively via other headers).
Fix testLargeTransfer deadlock — the 128KB write-then-read was sequential in a single coroutine, which deadlocks when the payload exceeds the kernel's TCP send buffer (smaller on FreeBSD). The fix runs the writer and reader as concurrent coroutines.

FreeBSD requires explicit -lpthread (pthreads lives in libthr.so, not
libc) and an explicit sys/socket.h include for sockaddr types (not
transitively included via netinet/in.h as on Linux).
Run writer and reader as concurrent coroutines to avoid deadlocking
when the 128KB payload exceeds FreeBSD's 32KB TCP send buffer.
@cppalliance-bot
Copy link

An automated preview of the documentation is available at https://154.corosio.prtest3.cppalliance.org/index.html

If more commits are pushed to the pull request, the docs will rebuild at the same URL.

2026-02-19 13:40:50 UTC

@codecov
Copy link

codecov bot commented Feb 19, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.26%. Comparing base (deba714) to head (cc0b500).
⚠️ Report is 2 commits behind head on develop.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #154      +/-   ##
===========================================
+ Coverage    82.23%   82.26%   +0.03%     
===========================================
  Files           70       70              
  Lines         5876     5876              
===========================================
+ Hits          4832     4834       +2     
+ Misses        1044     1042       -2     
Files with missing lines Coverage Δ
include/boost/corosio/detail/endpoint_convert.hpp 100.00% <ø> (ø)

... and 1 file with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update deba714...cc0b500. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@cppalliance-bot
Copy link

GCOVR code coverage report https://154.corosio.prtest3.cppalliance.org/gcovr/index.html
LCOV code coverage report https://154.corosio.prtest3.cppalliance.org/genhtml/index.html
Coverage Diff Report https://154.corosio.prtest3.cppalliance.org/diff-report/index.html

Build time: 2026-02-19 13:44:52 UTC

@sgerbino sgerbino merged commit 5c87bce into cppalliance:develop Feb 19, 2026
16 checks passed
@sgerbino sgerbino deleted the pr/freebsd branch February 19, 2026 13:51
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.

2 participants