Re: CVS commit: pkgsrc/devel/gtexinfo

["Jonathan Perkin" <jperkin%joyent.com@localhost>]

* On 2013-01-11 at 14:57 GMT, Sean Boudreau wrote:

> Module Name:  pkgsrc
> Committed By: seanb
> Date:         Fri Jan 11 14:57:27 UTC 2013
> 
> Modified Files:
>       pkgsrc/devel/gtexinfo: Makefile
> 
> Log Message:
> Don't use REPLACE_BASH in util/texi2dvi script.  /bin/sh seems fine
> in my testing.  Adding dependency on bash in gtexinfo causes a circular
> dependency on hosts without makeinfo since bash requires makeinfo and
> makeinfo gets replaced with gtexinfo.

This isn't adequate, not all systems' /bin/sh support the 'local'
keyword that this script relies upon, which is why this change was
made in the first place.  It appears to try to provide its own
version, but this still causes different behaviour:

  $ /bin/sh /opt/local/bin/texi2dvi 
  /opt/local/bin/texi2dvi[128]: local: not found [No such file or directory]
  /opt/local/bin/texi2dvi[209]: local: not found [No such file or directory]

  $ /bin/bash /opt/local/bin/texi2dvi
  /opt/local/bin/texi2dvi: Missing file arguments.
  /opt/local/bin/texi2dvi: Try `--help' for more information.

This is on SunOS 5.11, where /bin/sh is ksh93.

-- 
Jonathan Perkin  -  Joyent, Inc.  -  www.joyent.com