Re: CVS commit: src/tools/compat/sys
On Mar 27, 8:19am, christoph_eg...@gmx.de (Christoph Egger) wrote: -- Subject: Re: CVS commit: src/tools/compat/sys | On 26.03.10 22:34, Christos Zoulas wrote: | > In article <1933.1269630...@splode.eterna.com.au>, | > matthew green wrote: | >> | >> | >> tools/ctfconvert/../../external/cddl/osnet/sys/sys/time.h:71: error: | >> 'CLOCK_REALTIME' undeclared (first use in this function) | >> | >> | >> | >> shouldn't this be fixed with something in nbtool_config.h? | > | > Just back the whole thing out; I fixed it in the dtrace sys/time.h where it | > belongs. | | Done. See | http://mail-index.netbsd.org/source-changes/2010/03/27/msg008205.html | | Christoph Thanks! christos
Re: CVS commit: src/tools/compat/sys
On 26.03.10 22:34, Christos Zoulas wrote: > In article <1933.1269630...@splode.eterna.com.au>, > matthew green wrote: >> >> >> tools/ctfconvert/../../external/cddl/osnet/sys/sys/time.h:71: error: >> 'CLOCK_REALTIME' undeclared (first use in this function) >> >> >> >> shouldn't this be fixed with something in nbtool_config.h? > > Just back the whole thing out; I fixed it in the dtrace sys/time.h where it > belongs. Done. See http://mail-index.netbsd.org/source-changes/2010/03/27/msg008205.html Christoph
Re: CVS commit: src/tools/compat/sys
In article <1933.1269630...@splode.eterna.com.au>, matthew green wrote: > > > tools/ctfconvert/../../external/cddl/osnet/sys/sys/time.h:71: error: > 'CLOCK_REALTIME' undeclared (first use in this function) > > > >shouldn't this be fixed with something in nbtool_config.h? Just back the whole thing out; I fixed it in the dtrace sys/time.h where it belongs. christos
Re: CVS commit: src/tools/compat/sys
On Mar 26, 2010, at 12:00 PM, matthew green wrote: > > > tools/ctfconvert/../../external/cddl/osnet/sys/sys/time.h:71: error: > 'CLOCK_REALTIME' undeclared (first use in this function) > > > > shouldn't this be fixed with something in nbtool_config.h? That's what I was thinking since ctf on cross host isn't that useful.
re: CVS commit: src/tools/compat/sys
tools/ctfconvert/../../external/cddl/osnet/sys/sys/time.h:71: error: 'CLOCK_REALTIME' undeclared (first use in this function) shouldn't this be fixed with something in nbtool_config.h?
Re: CVS commit: src/tools/compat/sys
Original-Nachricht > Datum: Sat, 27 Mar 2010 03:03:06 +1100 > Von: matthew green > An: "Christoph Egger" > CC: source-changes-d@NetBSD.org > Betreff: re: CVS commit: src/tools/compat/sys > > >>toolchain buildfix on OSX with MKDTRACE=yes: >> >>The host doesn't provide CLOCK_REALTIME but >>ctf uses it. >>See > http://mail-index.netbsd.org/current-users/2010/03/20/msg012963.html >> >> this seems bogus to me. >> >> please only define it if it isn't defined, at least. > > >I tried the other way around before I posted the my fix >to the list: > >#include_next > >#ifndef CLOCK_REALTIME >#define CLOCK_REALTIME 0 >#endif > >But this failed the same way: CLOCK_REALTIME undeclared. > > hmmm, could you post the error? thanks. tools/ctfconvert/../../external/cddl/osnet/sys/sys/time.h:71: error: 'CLOCK_REALTIME' undeclared (first use in this function) >Note, ctf uses CLOCK_REALTIME in the same header where it >includes sys/time.h. > > BTW, i'm not sure that using #include_next is OK (it's gcc > specific) and i don't see it used elsewhere in a quick look. If you want to see that go away then ctf needs to be fixed first (which implies to wait for darran@ to be back from vacation in first place :)) See line 34 in ctf's sys/time.h (above file where error occured). Christoph
re: CVS commit: src/tools/compat/sys
>toolchain buildfix on OSX with MKDTRACE=yes: > >The host doesn't provide CLOCK_REALTIME but >ctf uses it. >See http://mail-index.netbsd.org/current-users/2010/03/20/msg012963.html > > this seems bogus to me. > > please only define it if it isn't defined, at least. I tried the other way around before I posted the my fix to the list: #include_next #ifndef CLOCK_REALTIME #define CLOCK_REALTIME 0 #endif But this failed the same way: CLOCK_REALTIME undeclared. hmmm, could you post the error? thanks. Note, ctf uses CLOCK_REALTIME in the same header where it includes sys/time.h. BTW, i'm not sure that using #include_next is OK (it's gcc specific) and i don't see it used elsewhere in a quick look. >No comments. > > try posting to the right mailing list. tech-toolchain would > be the right list for this sort of change. Thanks for the hint. np.
Re: re: CVS commit: src/tools/compat/sys
> >Module Name: src >Committed By: cegger >Date: Fri Mar 26 07:16:12 UTC 2010 > >Added Files: > src/tools/compat/sys: time.h > >Log Message: >toolchain buildfix on OSX with MKDTRACE=yes: > >The host doesn't provide CLOCK_REALTIME but >ctf uses it. >See http://mail-index.netbsd.org/current-users/2010/03/20/msg012963.html > > this seems bogus to me. > > please only define it if it isn't defined, at least. I tried the other way around before I posted the my fix to the list: #include_next #ifndef CLOCK_REALTIME #define CLOCK_REALTIME 0 #endif But this failed the same way: CLOCK_REALTIME undeclared. Note, ctf uses CLOCK_REALTIME in the same header where it includes sys/time.h. >No comments. > > try posting to the right mailing list. tech-toolchain would > be the right list for this sort of change. Thanks for the hint. Christoph
re: CVS commit: src/tools/compat/sys
Module Name: src Committed By:cegger Date:Fri Mar 26 07:16:12 UTC 2010 Added Files: src/tools/compat/sys: time.h Log Message: toolchain buildfix on OSX with MKDTRACE=yes: The host doesn't provide CLOCK_REALTIME but ctf uses it. See http://mail-index.netbsd.org/current-users/2010/03/20/msg012963.html this seems bogus to me. please only define it if it isn't defined, at least. No comments. try posting to the right mailing list. tech-toolchain would be the right list for this sort of change. .mrg.