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 <[email protected]> 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 <[email protected]> > 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 <[email protected]> >> 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) <[email protected]> >>> 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 >>>> [email protected] >>>> http://lists.linaro.org/mailman/listinfo/lng-odp >>>> >>>> >>> >>> >>> -- >>> *Mike Holmes* >>> Linaro Sr Technical Manager >>> LNG - ODP >>> >>> _______________________________________________ >>> lng-odp mailing list >>> [email protected] >>> http://lists.linaro.org/mailman/listinfo/lng-odp >>> >>> >> > > > -- > *Mike Holmes* > Linaro Sr Technical Manager > LNG - ODP > -- *Mike Holmes* Linaro Sr Technical Manager LNG - ODP
_______________________________________________ lng-odp mailing list [email protected] http://lists.linaro.org/mailman/listinfo/lng-odp
