On Wed, Mar 8, 2017 at 2:16 AM, <otr...@employees.org> wrote:

> Jon,
>
> I have also seen this error. What I suspect happens is that while pneum is
> already built and that the build dependency is correct, the library isn't
> installed under the install- directory yet.
>
> I didn't manage to reproduce it right now, but would you mind trying this
> to see if it works:
>
> diff --git a/src/vpp-api/python/Makefile.am b/src/vpp-api/python/Makefile.
> am
> index 5407682..c60e4f4 100644
> --- a/src/vpp-api/python/Makefile.am
> +++ b/src/vpp-api/python/Makefile.am
> @@ -39,7 +39,7 @@ libpneum_la_LDFLAGS = -module
>  libpneum_la_CPPFLAGS =
>
>  # TODO: Support both Python 2 and 3.
> -install-exec-local:
> +install-exec-local: install-libLTLIBRARIES
>         cd $(srcdir);                                                   \
>         mkdir -p $(pythondir);                                          \
>         mkdir -p $(pyexecdir);                                          \
>
>
> (In other news I am looking at using CFFI instead, and then the whole
> vpp_api.so would go away).
>
> Cheers,
> Ole
>

Ole,

I am no fairly confident that this patch fixes the race condition issue
that I was seeing.

At first, I was seeing a failure at about a 1 in 3 rate.  That backed off
to about 1 in 9 recently.  No idea why...

However, after applying this patch, I've run more than 30 builds
successfully without a failure now.

I've not done the statistics to determine how confident I am. :-)
But I say go with it!   I'll watch for it and let you know if it resurfaces!

Thank you!
jdl
_______________________________________________
vpp-dev mailing list
vpp-dev@lists.fd.io
https://lists.fd.io/mailman/listinfo/vpp-dev

Reply via email to