Attention is currently required from: Timur Davydov.

pespin has posted comments on this change by Timur Davydov. ( 
https://gerrit.osmocom.org/c/osmo-trx/+/42654?usp=email )

Change subject: build: detect pthread_setname_np and sched_* APIs, guard usage
......................................................................


Patch Set 2: Code-Review+1

(1 comment)

File CommonLibs/Threads.cpp:

https://gerrit.osmocom.org/c/osmo-trx/+/42654/comment/5885e203_eb006d70?usp=email
 :
PS1, Line 39:
> Good catch, thanks. I’ll add the missing #include "config.h" to all files 
> with HAVE_ stuff is added.
In general config.h has to be set as the first include in the file, so it sets 
stuff which in turn can be used by other headers.



--
To view, visit https://gerrit.osmocom.org/c/osmo-trx/+/42654?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings?usp=email

Gerrit-MessageType: comment
Gerrit-Project: osmo-trx
Gerrit-Branch: master
Gerrit-Change-Id: Ic3f1492544439d88c5e6683330c6696de1c2f85b
Gerrit-Change-Number: 42654
Gerrit-PatchSet: 2
Gerrit-Owner: Timur Davydov <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <[email protected]>
Gerrit-Attention: Timur Davydov <[email protected]>
Gerrit-Comment-Date: Thu, 30 Apr 2026 15:28:32 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Comment-In-Reply-To: pespin <[email protected]>
Comment-In-Reply-To: Timur Davydov <[email protected]>

Reply via email to