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  m...@eterna.com.au wrote:
   
   copy  paste from above url
   tools/ctfconvert/../../external/cddl/osnet/sys/sys/time.h:71: error:
   'CLOCK_REALTIME' undeclared (first use in this function)
   /copy  paste


 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-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  m...@eterna.com.au wrote:
|
|copy  paste from above url
|tools/ctfconvert/../../external/cddl/osnet/sys/sys/time.h:71: error:
|'CLOCK_REALTIME' undeclared (first use in this function)
|/copy  paste
| 
| 
|  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-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 sys/time.h 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.


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 sys/time.h 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 sys/time.h

#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

   
   toolchain buildfix on OSX with MKDTRACE=yes:
   
   The host sys/time.h 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 sys/time.h
   
   #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: 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 m...@eterna.com.au
 An: Christoph Egger christoph_eg...@gmx.de
 CC: source-changes-d@NetBSD.org
 Betreff: re: CVS commit: src/tools/compat/sys

 

toolchain buildfix on OSX with MKDTRACE=yes:

The host sys/time.h 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 sys/time.h

#ifndef CLOCK_REALTIME
#define CLOCK_REALTIME 0
#endif

But this failed the same way: CLOCK_REALTIME undeclared.
 
 hmmm, could you post the error?  thanks.

copy  paste from above url
tools/ctfconvert/../../external/cddl/osnet/sys/sys/time.h:71: error:
'CLOCK_REALTIME' undeclared (first use in this function)
/copy  paste
   
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
   
   copy  paste from above url
   tools/ctfconvert/../../external/cddl/osnet/sys/sys/time.h:71: error:
   'CLOCK_REALTIME' undeclared (first use in this function)
   /copy  paste


shouldn't this be fixed with something in nbtool_config.h?


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

2010-03-26 Thread Matt Thomas

On Mar 26, 2010, at 12:00 PM, matthew green wrote:

 
   copy  paste from above url
   tools/ctfconvert/../../external/cddl/osnet/sys/sys/time.h:71: error:
   'CLOCK_REALTIME' undeclared (first use in this function)
   /copy  paste
 
 
 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 Christos Zoulas
In article 1933.1269630...@splode.eterna.com.au,
matthew green  m...@eterna.com.au wrote:
   
   copy  paste from above url
   tools/ctfconvert/../../external/cddl/osnet/sys/sys/time.h:71: error:
   'CLOCK_REALTIME' undeclared (first use in this function)
   /copy  paste


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