On Wed, Nov 28, 2018 at 09:22:13PM +0000, Jason Gunthorpe wrote: > On Wed, Nov 28, 2018 at 03:57:08PM +0200, Leon Romanovsky wrote: > > diff --git a/buildlib/travis-build b/buildlib/travis-build > > index c241eb06..c8c3e539 100755 > > +++ b/buildlib/travis-build > > @@ -9,7 +9,7 @@ mkdir build-travis build32 build-sparse build-aarch64 > > > > # Build with latest clang first > > cd build-travis > > -CC=clang-7 CFLAGS=-Werror cmake -GNinja .. -DIOCTL_MODE=both -DENABLE_STATIC=1 > > +CC=clang-7 CFLAGS=-Werror cmake -GNinja .. -DIOCTL_MODE=both -DENABLE_STATIC=1 -DNO_PYVERBS=1 > > ninja > > ../buildlib/check-build --src .. --cc clang-7 > > Now that we have a Senial base image we shouldn't need NO_PYVERBS anymore? Last time, when I wrote this patch, cython produced C files, which were not good enough for "CC=clang-7", this is why I disabled back then. I'll recheck, maybe my changes in compilation flags fixed those failures. Thanks > > Jason
Attachment:
signature.asc
Description: PGP signature
