Re: CVS commit: pkgsrc/audio/sptk

[David Holland <dholland-pkgchanges%netbsd.org@localhost>]

On Tue, Jul 05, 2011 at 06:09:59PM +0200, Alistair Crooks wrote:
 > On Sun, Jun 19, 2011 at 05:03:44PM +0000, David Holland wrote:
 > > On Thu, Jun 09, 2011 at 09:36:36AM +0900, OBATA Akio wrote:
 > >  >> > * exactly depend on csh, some installed scripts are csh script.
 > >  >>
 > >  >> Does USE_TOOLS+=csh:run work now? I thought there wasn't support for
 > >  >> csh in TOOLS.
 > >  > 
 > >  > % make show-depends TOOLS_PLATFORM.csh=
 > >  > tcsh-[0-9]*:../../shells/tcsh
 > >  > libX11>=1.1:../../x11/libX11
 > >  > 
 > >  > http://mail-index.netbsd.org/pkgsrc-changes/2005/06/24/0103.html
 > > 
 > > Huh. That makes radiance a lot easier to deal with. The whole issue
 > > there was that agc didn't want to add TOOLS support for csh :-)
 > 
 > Yeah, I'm still not convinced that was me - I'm usually fairly good
 > with conversations I've had, and that one doesn't stick in my mind at
 > all (to say nothing of the fact that I'm a happy tcsh user anyway :-))

Hrm... well, it's what I wrote in my commit message and I don't
*think* I would have gotten it wrong, or, given that it was apparently
back in 2008 and I was very new at pkgsrc hacking then, launched on a
quixotic mission like rewriting csh scripts in sh without permission.

It's also possible that I've forgotten what the actual problem was.
It obviously had *something* to do with adding more support for csh
scripts. and csh scripts should definitely not be encouraged. :-)
(and yes, I use tcsh too)

anyhow, it's time I went and fixed radiance.

-- 
David A. Holland
dholland%netbsd.org@localhost