Re: pkg/38837 (CVS commit: pkgsrc/lang/clisp)

[Aleksej Saushev <asau%hotbox.ru@localhost>]

The following reply was made to PR pkg/38837; it has been noted by GNATS.

From: Aleksej Saushev <asau%hotbox.ru@localhost>
To: gnats-bugs%gnats.NetBSD.org@localhost
Cc: 
Subject: Re: pkg/38837 (CVS commit: pkgsrc/lang/clisp)
Date: Tue, 15 Jul 2008 21:26:21 +0400

 David Holland <dholland-pkgchanges%netbsd.org@localhost> writes:
 
 > On Tue, Jul 15, 2008 at 01:32:54PM +0000, Adam Hoka wrote:
 >  > Update to CLISP 2.46.
 >
 > Is this likely to fix PR 38837, about bashisms in the configure script?
 
 Probably, I've missed this PR during my periodic raids...
 
 I've read PR several times and don't find any problem:
 1. lang/clisp does not show up in bulk builds, regularly run by
 Joerg on this very platform.
 2. lang/clisp does not show up in bulk builds for rather long time.
 3. The logic in configure does use bashisms to restart with bash
 after detection of broken /bin/sh (Solaris). I don't see how this
 breaks build, when sane shell is used, there're no indication to
 that, only assumption.
 
 Please, provide more information on how it actually breaks the build.