distro diversity in autobuilds

Martin Schwenke martin at meltin.net
Thu Mar 22 02:37:48 UTC 2018


On Thu, 22 Mar 2018 13:43:47 +1300, Douglas Bagnall via samba-technical
<samba-technical at lists.samba.org> wrote:

> On 22/03/18 12:07, Douglas Bagnall via samba-technical wrote:
> > From time to time, Andreas says something like:  
> >> as I'm probably the only one who actually runs our tests on a
> >> different distro than Ubuntu, [...]  
> > 
> > Which, along with recent Python 2.6 regressions, has prompted me to
> > try running something else on the Catalyst cloud. And after much
> > tweaking, we can successfully run... {drum-roll}... Debian!
> > (Debian 9/stretch/stable to be precise).
> > 
> > But I also have plans for Centos 6.6 and 7.
> > 
> > In Centos 6.6, I arrive here in samba.stderr:
> > 
> > [ 671/4143] Compiling lib/util/become_daemon.c
> > cc1: warnings being treated as errors
> > ../lib/util/become_daemon.c: In function ‘daemon_ready’:
> > ../lib/util/become_daemon.c:128: error: declaration of ‘daemon’ shadows
> > a global declaration
> > /usr/include/unistd.h:956: error: shadowed declaration is here
> > 
> > GCC is "gcc (GCC) 4.4.7 20120313 (Red Hat 4.4.7-18)".
> > 
> > Commit a50092531af8c837c6e45c3c755fcb20b6ce323b looks culpable.  
> 
> So I fixed this one (attached) but now it fails on a similar thing a
> moment later:
> 
> [ 701/4151] Compiling lib/util/tini.c
> cc1: warnings being treated as errors
> ../lib/util/tfork.c: In function ‘tfork_status’:
> ../lib/util/tfork.c:809: error: declaration of ‘wait’ shadows a global declaration
> /usr/include/sys/wait.h:116: error: shadowed declaration is here
> Waf: Leaving directory `/home/centos/autobuild/b16891/samba/bin'
> Build failed:  -> task failed (err #1): 
> 	{task: cc tfork.c -> tfork_32.o}
> make: *** [all] Error 1
> 
> which suggests I should just find a way of turning off this silly
> warning. Therefore I NACK my own patch and recommend you do the same
> unless you think gcc 4.4.7 was onto something important.

I broke it, the compiler is stupid, we should fix it...

Reviewed-by: Martin Schwenke <martin at meltin.net>

Provided, if course, you nix your own NACK.  ;-)

peace & happiness,
martin



More information about the samba-technical mailing list