Re: CVS commit: src/tools/compat/sys

2010-03-27 Thread Christos Zoulas
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

2010-03-27 Thread Christoph Egger
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

2010-03-26 Thread Christos Zoulas
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

2010-03-26 Thread Matt Thomas

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

2010-03-26 Thread matthew green
   
   
   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

2010-03-26 Thread Christoph Egger

 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

2010-03-26 Thread matthew green

   
   >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

2010-03-26 Thread Christoph Egger

> 
>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

2010-03-26 Thread matthew green

   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.