Menu ▾ ▴

#1969 Use PTHREAD_RECURSIVE_MUTEX_INITIALIZER_NP if libc defines it

3.8.0
closed-fixed
Hemantha Beecherla
IPMI Direct plugin (99)
IPMI Direct plugin
5
2018-02-28
2017-09-15
Khem Raj
No

Currently we check for OS before using PTHREAD_RECURSIVE_MUTEX_INITIALIZER_NP but it would be better to check for it being defined rather.

This helps in fixing build with musl on linux where PTHREAD_RECURSIVE_MUTEX_INITIALIZER_NP is not available but its running on Linux. Current assumption is that its universally available on Linux based OSes

2 Attachments
0001-plugins-Check-for-PTHREAD_RECURSIVE_MUTEX_INITIALIZE.patch
1969.patch

Discussion

  • Hemantha Beecherla

    Hemantha Beecherla - 2018-01-11
    • status: open --> closed-fixed
    • assigned_to: Hemantha Beecherla
    • Attachments has changed:

    Diff:

    --- old
+++ new
@@ -1 +1,2 @@
 0001-plugins-Check-for-PTHREAD_RECURSIVE_MUTEX_INITIALIZE.patch (1.6 kB; text/x-patch)
+1969.patch (1.2 kB; application/octet-stream)
     

  • Hemantha Beecherla

    Hemantha Beecherla - 2018-01-11

    Fixed in revision 7732.
    Attached commited patch to the bug.

     

  • dr_mohan

    dr_mohan - 2018-01-17
    • 3.7.0: Future --> 3.8.0
     

  • Hemantha Beecherla

    Hemantha Beecherla - 2018-02-28
    • labels: --> IPMI Direct plugin