Guys, I’ll take a look at e.g. gerrit 9440, ip_frag.c and see if I can fix it.
Under the circumstances, it seems perfectly OK to s/ON/OFF/ as needed in the per-file patch verification on/off switch: /* * fd.io coding-style-patch-verification: ON * * Local Variables: * eval: (c-set-style "gnu") * End: */ Thanks… Dave From: vpp-dev-boun...@lists.fd.io [mailto:vpp-dev-boun...@lists.fd.io] On Behalf Of Marco Varlese Sent: Friday, December 15, 2017 5:16 AM To: Damjan Marion (damarion) <damar...@cisco.com> Cc: vpp-dev <vpp-dev@lists.fd.io> Subject: Re: [vpp-dev] openSUSE build fails Hi Damjan, On Fri, 2017-12-15 at 09:06 +0000, Damjan Marion (damarion) wrote: On 15 Dec 2017, at 08:52, Marco Varlese <mvarl...@suse.de<mailto:mvarl...@suse.de>> wrote: Damjan, On Thu, 2017-12-14 at 16:04 +0000, Damjan Marion (damarion) wrote: Folks, I'm hearing from multiple people that OpenSUSE verify job is failing (again). I haven't heard (or read) anything over the mailing list otherwise I would have looked into it. Also, if you hear anything like that you can always ping me directly and I will look into it... yes, people pinging me... See https://gerrit.fd.io/r/#/c/9440/ also: https://gerrit.fd.io/r/#/c/9813/ - abandoned but it shows that something was wrong Ok, so just summarizing our conversation on IRC for others too. That issue is connected to the different versions of INDENT (C checkstyle) installed on the different distros. openSUSE runs 2.2.10 whilst CentOS and Ubuntu run 2.2.11 What strikes me is that the upstream repo https://ftp.gnu.org/gnu/indent/ has 2.2.10 as last revision. Our indent package maintainer is looking at possible other sources where Indent could "live" these days and will let me know as soon as she finds out. @Thomas Herbert, would you know the source where the Indent package on CentOS come from? Maybe that could help... So generally speaking i would like to question having verify jobs for multiple distros. Is there really a value in compiling same code on different distros. Yes I know gcc version can be different, but that can be addressed in simpler way, if it needs to be addressed at all. More distros means more moving parts and bigger chance that something will fail. Well, I am not sure how to interpret this but (in theory) a build should be reproducible in the first place and I should not worry about problems with build outcomes. It doesn't only affect openSUSE and I raised it many times over the mailing-list; when you need to run "recheck" multiple times to have a build succeed. IMHO the issue should be addressed and not solved by putting it under the carpet... We all know that we have extreme fragile system, as obviously we are not be able to fix that in almost 2 years, so as long as the system is as it increasing complexity doesn't help and just causes frustration. Also it cost resources.... That is a different matter and if that's the case then it should be discussed seriously; raising this argument now, after having had people investing their times in getting stuff up and running isn't really a cool thing... Marco, decision to have verify jobs on 2 distros was made much before you joined the project, and I don't remember serious decision on that topic, it might be that at that time we were simply unexperienced, or maybe we didn't expect infra to be so fragile. Fact is that now we have ridiculous situation, 2 verify jobs says patch is OK, 3rd one says it is not. Which one to trust? So please don't take this personal, i know you invested time to get suse build working, but still I think it is a valid question to ask, do we really need 3 verify jobs. Should we have 4 tomorrow if somebody invest his time to do verify job on Archlinux for example? Thanks, Damjan -- Marco V SUSE LINUX GmbH | GF: Felix Imendörffer, Jane Smithard, Graham Norton HRB 21284 (AG Nürnberg) Maxfeldstr. 5, D-90409, Nürnberg
_______________________________________________ vpp-dev mailing list vpp-dev@lists.fd.io https://lists.fd.io/mailman/listinfo/vpp-dev