[PATCH] ctdb/tests: add missing dependency to libreplace for PTHREAD_MUTEX_ROBUST

Amitay Isaacs amitay at gmail.com
Fri Feb 8 03:36:42 UTC 2019


On Fri, Feb 8, 2019 at 2:28 PM Amitay Isaacs <amitay at gmail.com> wrote:
>
> On Thu, Feb 7, 2019 at 10:16 PM Stefan Metzmacher via samba-technical
> <samba-technical at lists.samba.org> wrote:
> >
> > Hi Björn,
> >
> > > The attached path solves a build issue in the ctdb tests:
> > > [2694/3871] Compiling ctdb/tests/src/test_mutex_raw.c
> > > ../ctdb/tests/src/test_mutex_raw.c: In function ‘main’:
> > > ../ctdb/tests/src/test_mutex_raw.c:332: error: ‘PTHREAD_MUTEX_ROBUST’
> > > undeclared (first use in this function)
> > >
> > > Please review :-)
> >
> > I think we want deps='replace pthread'.
> >
> > Otherwise it looks good.
>
> NACK.  This test is in ctdb directory just to keep track of it.  It
> has nothing to do with samba or ctdb.
>
> It's better to keep this test standalone and not pollute with build
> dependencies on samba tree.
>
> Björn, what platform is the build failing on?

Looking at the history of changes, I see that I added replace to avoid
using _np versions of pthread functions.  And later they were again
removed to make it standalone.

Since this test has nothing to do with samba/ctdb, but related to bug
in glibc on linux, I would just drop the build rule for
test_mutex_raw.c.

Amitay.



More information about the samba-technical mailing list