On 9 December 2014 at 12:57, Bill Fischofer <bill.fischo...@linaro.org> wrote:
> If we can get a complete list of which files currently have issues I'm > thinking that would be an excellent list to mull over during the Christmas > break. > For a list just add the cflags line "./configure CFLAGS=-std=c99" and then make -k 2>&1 | grep "^.*\.c:" Api is currently odp_packet_io.c odp_shared_memory.c odp_timer.c There are more in the examples For those of us who will be "off" we may still want a coding fix for an > hour or so. :) > > Also it's an excellent way for those on the ODP mailing list who want to > "get their hands dirty" with ODP code to tackle something small and focused. > > Bill > > On Tue, Dec 9, 2014 at 11:54 AM, Mike Holmes <mike.hol...@linaro.org> > wrote: > >> I think adding it explicitly when using the patch checking tool >> "apply-and-build.sh" might be a way forward. >> >> If we can get agreement on C99, then bugzilla already has many of these >> bugs listed, >> <https://bugs.linaro.org/buglist.cgi?component=General&list_id=3080&product=OpenDataPlane&resolution=---> >> including the source of Robbies issue. All we need is agreement that C99 is >> our direction and git blame will show us the likely best person to fix each >> issue. >> >> On 9 December 2014 at 12:48, Bill Fischofer <bill.fischo...@linaro.org> >> wrote: >> >>> OK, I understand this is perhaps a longer-term project, but is there a >>> file-level way we can enable this check? If yes, then we can ask that as >>> part of modules that are currently being patched that c99 checks be >>> included in them. That way files will be brought into compliance and then >>> stay in compliance after they've been merged. >>> >>> On Tue, Dec 9, 2014 at 11:40 AM, Mike Holmes <mike.hol...@linaro.org> >>> wrote: >>> >>>> sorry, I cleaned the history try this: >>>> >>>> https://ci.linaro.org/view/odp-ci/job/odp-api-check-native-c99/buildhw=x86_64,label=build/16/console >>>> >>>> On 9 December 2014 at 12:39, Mike Holmes <mike.hol...@linaro.org> >>>> wrote: >>>> >>>>> Make -k gives the attached so it is easy to look past the first issue. >>>>> We cant make it default because everything breaks, we should fix >>>>> things and then the CI job which >>>>> has been building it this way for a while will eventually pass. If we >>>>> can get consensus on linux-generic sticking to to C99 it is worth >>>>> following >>>>> up on this build. >>>>> >>>>> The builds are for both ARM and X86, I had not been following it >>>>> becasue we did not have consensus on the C99 issue: X86 is working but Arm >>>>> filesystem needs curl installing. >>>>> >>>>> >>>>> https://ci.linaro.org/view/odp-ci/job/odp-api-check-native-c99/buildhw=x86_64,label=build/14/console >>>>> >>>>> We dont want to hijack the need to have ./configure check for the >>>>> version we are using however. >>>>> >>>>> >>>>> On 9 December 2014 at 12:17, Bill Fischofer <bill.fischo...@linaro.org >>>>> > wrote: >>>>> >>>>>> I suggest we turn on that by default. It's the one way to ensure >>>>>> that the issues will get fixed. Is this a stop-on-first-error situation >>>>>> or >>>>>> can we have it carry on so we get a complete list of what the c99 issues >>>>>> are? >>>>>> >>>>>> On Tue, Dec 9, 2014 at 11:02 AM, Mike Holmes <mike.hol...@linaro.org> >>>>>> wrote: >>>>>> >>>>>>> This might also be the right time to adhere to the use of only C99 >>>>>>> in linux-generics implementation ? >>>>>>> >>>>>>> CFLAGS=-std=c99 ./configure >>>>>>> make >>>>>>> >>>>>>> Throws up other issues, the first is in packet_io >>>>>>> >>>>>>> mike@fedora1:~/git/odp$ make >>>>>>> Making all in platform >>>>>>> make[1]: Entering directory '/home/mike/git/odp/platform' >>>>>>> Making all in linux-generic >>>>>>> make[2]: Entering directory >>>>>>> '/home/mike/git/odp/platform/linux-generic' >>>>>>> CC odp_packet_io.lo >>>>>>> odp_packet_io.c: In function 'odp_pktio_set_mtu': >>>>>>> odp_packet_io.c:512:35: error: 'caddr_t' undeclared (first use in >>>>>>> this function) >>>>>>> ret = ioctl(sockfd, SIOCSIFMTU, (caddr_t)&ifr); >>>>>>> ^ >>>>>>> odp_packet_io.c:512:35: note: each undeclared identifier is reported >>>>>>> only once for each function it appears in >>>>>>> Makefile:560: recipe for target 'odp_packet_io.lo' failed >>>>>>> >>>>>>> >>>>>>> >>>>>>> Mike >>>>>>> >>>>>>> On 9 December 2014 at 11:48, Robbie King (robking) < >>>>>>> robk...@cisco.com> wrote: >>>>>>> >>>>>>>> After cloning tip just now, I found that I couldn’t build the >>>>>>>> fresh >>>>>>>> >>>>>>>> workspace due to having GCC 4.6.3 (the C11 changes to the atomics >>>>>>>> >>>>>>>> bumps minimum GCC up to 4.8 as best I can tell). I’m not very >>>>>>>> familiar >>>>>>>> >>>>>>>> with what “./configure” can and can’t do, but it seems we should >>>>>>>> verify >>>>>>>> >>>>>>>> the compiler supports these constructs and fail during the configure >>>>>>>> >>>>>>>> phase (as opposed to build time). >>>>>>>> >>>>>>>> >>>>>>>> >>>>>>>> Thanks, >>>>>>>> >>>>>>>> Robbie >>>>>>>> >>>>>>>> _______________________________________________ >>>>>>>> lng-odp mailing list >>>>>>>> lng-odp@lists.linaro.org >>>>>>>> http://lists.linaro.org/mailman/listinfo/lng-odp >>>>>>>> >>>>>>>> >>>>>>> >>>>>>> >>>>>>> -- >>>>>>> *Mike Holmes* >>>>>>> Linaro Sr Technical Manager >>>>>>> LNG - ODP >>>>>>> >>>>>>> _______________________________________________ >>>>>>> lng-odp mailing list >>>>>>> lng-odp@lists.linaro.org >>>>>>> http://lists.linaro.org/mailman/listinfo/lng-odp >>>>>>> >>>>>>> >>>>>> >>>>> >>>>> >>>>> -- >>>>> *Mike Holmes* >>>>> Linaro Sr Technical Manager >>>>> LNG - ODP >>>>> >>>> >>>> >>>> >>>> -- >>>> *Mike Holmes* >>>> Linaro Sr Technical Manager >>>> LNG - ODP >>>> >>> >>> >> >> >> -- >> *Mike Holmes* >> Linaro Sr Technical Manager >> LNG - ODP >> > > -- *Mike Holmes* Linaro Sr Technical Manager LNG - ODP
_______________________________________________ lng-odp mailing list lng-odp@lists.linaro.org http://lists.linaro.org/mailman/listinfo/lng-odp