re: CVS commit: src/usr.bin/ktruss

2012-03-13 Thread matthew green
> On Tue, 13 Mar 2012, David Laight wrote: > >> h, host sort(1) dependancy. perhaps we need a TOOL_SORT. > > > >Posix deprected the +n terms (even our sort hacks argv[]). > >Shame, because they are what my fingers know about. > > I haven't yet seen a problem report that indicates a need for

Re: CVS commit: src/usr.bin/ktruss

2012-03-13 Thread Alan Barrett
On Tue, 13 Mar 2012, David Laight wrote: h, host sort(1) dependancy. perhaps we need a TOOL_SORT. Posix deprected the +n terms (even our sort hacks argv[]). Shame, because they are what my fingers know about. I haven't yet seen a problem report that indicates a need for a TOOL_SORT. Un

Re: CVS commit: src/usr.bin/ktruss

2012-03-13 Thread David Laight
On Tue, Mar 13, 2012 at 09:56:29AM +1100, matthew green wrote: > > > Module Name:src > > Committed By: dyoung > > Date: Mon Mar 12 22:02:07 UTC 2012 > > > > Modified Files: > > src/usr.bin/ktruss: makeerrnos.sh > > > > Log Message: > > Use 'sort -n -k 3' instead o

re: CVS commit: src/usr.bin/ktruss

2012-03-12 Thread matthew green
> Module Name: src > Committed By: dyoung > Date: Mon Mar 12 22:02:07 UTC 2012 > > Modified Files: > src/usr.bin/ktruss: makeerrnos.sh > > Log Message: > Use 'sort -n -k 3' instead of 'sort -n +2' since the former is more > portable. The latter is not supported by Mac OS X Lion.

Re: CVS commit: src/usr.bin/ktruss

2009-12-13 Thread Masao Uebayashi
> > Modified Files: > > src/usr.bin/ktruss: Makefile > > - AWK=${TOOL_AWK:Q} CPP=${CPP:Q} CPPFLAGS=${CPPFLAGS:Q} \ > > + ${GENCMD} ${.TARGET} \ > > + AWK="${TOOL_AWK:Q}" CPP="${CPP:Q}" CPPFLAGS="${CPPFLAGS:Q}" \ > > The extra quotes here look wrong. ${VAR:Q} should do the correct > amou

Re: CVS commit: src/usr.bin/ktruss

2009-12-13 Thread Alan Barrett
On Fri, 11 Dec 2009, Masao Uebayashi wrote: > Modified Files: > src/usr.bin/ktruss: Makefile > - AWK=${TOOL_AWK:Q} CPP=${CPP:Q} CPPFLAGS=${CPPFLAGS:Q} \ > + ${GENCMD} ${.TARGET} \ > + AWK="${TOOL_AWK:Q}" CPP="${CPP:Q}" CPPFLAGS="${CPPFLAGS:Q}" \ The extra quotes here look wrong.