yes…it was considered a trivial rebase….

IF that was the case then ill take the wrap for that one wrt it getting a +1…

in a nutshell..  patch 2 failed…got its -1 ok fine.

someone came and did a trivial rebase…IF those last three jobs  docs, csit-virl 
and whatever were also not set to run on trivial rebase
the -1 would have remained.   BUT since those jobs triggered and passed 
(csit-virl passed since thats built/run for ubuntu) it got a +1
based on that.

I have no idea why the merge happened though….


On Feb 19, 2018, at 6:37 PM, Florin Coras 
<<>> wrote:

That’s .. interesting. Patch set 3 didn’t run all the jobs. Any idea why that 
would happen?

Instead of reverting the patch, here’s a hammer-based attempt at solving the 
problem [1]. Damjan, feel free to propose a prettier solution :)



On Feb 19, 2018, at 5:09 PM, Ed Kern <<>> 

On Feb 19, 2018, at 2:47 PM, Jon Loeliger 
<<>> wrote:

Hey hey hey,

I tried to build a variant of vppsb and failed miserably
for lack of the __mm_testz_si128 symbol.

Wont make you feel better but my centos builds have been rolling and croaking 
over this for a few hours as well...

From whence does that arrive on my doorstep?

my best guess is that it came in with


Then again im still scratching my head on how this even got in since this 
failed verification on patch 2.
Then im guessing it got the +1 because it was deemed a trivial rebase so only a 
subset of jobs (aka not centos) fired.

But even the merge job for centos failed…and I thought (obviously incorrectly) 
that those had to have success for the
merge to take place.


I'm trying to figure out if vppsb needs to #define something new,
or #include something first, or if my system is suddenly missing
something assumed to be present and VPP cleverly allowed for
it to be missing while vppsb was less so?


```libtool: compile:  gcc -DPACKAGE_NAME=\"netlink\" 
-DPACKAGE_TARNAME=\"netlink\" "-DPACKAGE_VERSION=\"1.0.1-21~g888408a\"" 
"-DPACKAGE_STRING=\"netlink 1.0.1-21~g888408a\"" -DPACKAGE_BUGREPORT=\"\" 
-DLT_OBJDIR=\".libs/\" -DPACKAGE=\"netlink\" "-DVERSION=\"1.0.1-21~g888408a\"" 
-I. -Wall -fstack-protector -fPIC -Werror -g -DFORTIFY_SOURCE=2 -O2 
 -O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions 
-fstack-protector-strong --param=ssp-buffer-size=4 -grecord-gcc-switches -m64 
-mtune=generic -c librtnl/netns.c  -fPIC -DPIC -o librtnl/.libs/netns.o
In file included from 
                 from ./librtnl/netns.h:19,
                 from librtnl/netns.c:16:
 In function 'u8x16_is_all_zero':
 error: implicit declaration of function '_mm_testz_si128' 
   return _mm_testz_si128 ((__m128i) x, (__m128i) x);
cc1: all warnings being treated as errors

Reply via email to