cvs commit: src/bin/sh eval.c

[Martin Cracauer]

cracauer    2004/01/28 10:22:19 PST

  FreeBSD src repository

  Modified files:
    bin/sh               eval.c 
  Log:
  The PR diff I committed recently had one bug noticed by
  Joe Marcus Clarke <marcus at FreeBSD.ORG>, subshells could lose a
  non-zero exit status.
  
  This commit is Joe's proposed patch.  Thanks!
  
  I verified that the problem Joe found is fixed and I ran a full world
  with this patch.
  
  I don't plan to ever commit language patches to /bin/sh again.  It is
  a minefield too big to navigate without a full-time committment, which
  I am not willing to do on our /bin/sh.
  
  Under normal circumstances I would recommend using NetBSD's sh which
  has a lot of language fixes (like the ones what these patches were
  about) but unfortunately they had implemented broken signal behaviour
  for shellscript containing interactive programs.  Similar issues apply
  to pdksh which is OpenBSD's sh.
  
  From my perspective bash2 is the only really working bourne sh out
  there and that one is GPLed.  Oh well.
  
  Revision  Changes    Path
  1.41      +2 -1      src/bin/sh/eval.c