[PATCH] torture: Fix the O3 developer build

[Michael Adam]

On 2016-03-22 at 13:38 -0700, Jeremy Allison wrote:
> On Tue, Mar 22, 2016 at 06:41:42PM +0100, Michael Adam wrote:
> > On 2016-03-22 at 17:20 +0100, Volker Lendecke wrote:
> > > On Tue, Mar 22, 2016 at 05:05:55PM +0100, Michael Adam wrote:
> > > > On 2016-03-22 at 08:50 -0700, Jeremy Allison wrote:
> > > > > On Tue, Mar 22, 2016 at 04:05:50PM +0100, Volker Lendecke wrote:
> > > > > > Hi!
> > > > > > 
> > > > > > Apologies for the repeated mails in this regard: Is anybody on this list
> > > > > > still interested in this kind of spam, or do you want me to keep them
> > > > > > local in my tree?
> > > > > 
> > > > > Yes, yes 1000x yes - please keep them coming :-).
> > > > 
> > > > Indeed... Jeremy was again faster... :-)
> > > > 
> > > > I guess the actual message (instead of apologies) was
> > > > "When will you guys finally be careful enough with the code
> > > > you write, so that I don't have to fix up behind you?..."
> > > 
> > > Right. The question really is -- if I'm the only one compiling with
> > > -O3 for the additional warnings, the Samba Team as a whole does not
> > > see this as a priority.
> > 
> > We are all just lazy farts, and not implicitly neglcting -O3 but
> > relying on you... I propose that we add a -O3 build target
> > to our autobuild, so that there are no excuses.
> > 
> > See the attached patch.
> > 
> > Note that this fails on my f23 box... need to fix a few
> > O3 bugs as it seems ... :-)
> 
> Let's fix those as well, then push the whole thing :-).

Ok ... Now this updated patch survices autobuild on my box.

Changes:

- Volker's previous patch to torture/smb2/replay.c was applied first :-)

- I had to add "--abi-check-disable" to configure to survice the
  build -- not sure if abi check is expected to fail if building
  with -O3 ?

  ==> have added this as a separate patch for clarity. can be
  squashed if needed. will run test on sn-devel-144 for
  verification.

- I have modified the original patch to not run make -j but just
  make

Cheers - Michael
-------------- next part --------------
From 45c50454912e643672bccdc8eb6492b49c209e66 Mon Sep 17 00:00:00 2001
From: Michael Adam <obnox at samba.org>
Date: Tue, 22 Mar 2016 22:02:12 +0100
Subject: [PATCH] autobuild: skip abi check for -O3 build

Signed-off-by: Michael Adam <obnox at samba.org>
---
 script/autobuild.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/script/autobuild.py b/script/autobuild.py
index 1c172b2..886d227 100755
--- a/script/autobuild.py
+++ b/script/autobuild.py
@@ -82,7 +82,7 @@ tasks = {
 
     # test build with -O3 -- catches extra warnings and bugs
     "samba-o3" : [ ("random-sleep", "../script/random-sleep.sh 60 600", "text/plain"),
-                   ("configure", "ADDITIONAL_CFLAGS='-O3' ./configure.developer --with-selftest-prefix=./bin/ab" + samba_configure_params, "text/plain"),
+                   ("configure", "ADDITIONAL_CFLAGS='-O3' ./configure.developer --with-selftest-prefix=./bin/ab --abi-check-disable" + samba_configure_params, "text/plain"),
                    ("make", "make", "text/plain"),
                    ("test", "make quicktest FAIL_IMMEDIATELY=1 TESTS='\(ad_dc\)'", "text/plain"),
                    ("install", "make install", "text/plain"),
-- 
2.5.5

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20160322/3582b1df/signature.sig>