NetBSD-Bugs archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

PR/48489 CVS commit: src/bin/sh



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

From: "Robert Elz" <kre%netbsd.org@localhost>
To: gnats-bugs%gnats.NetBSD.org@localhost
Cc: 
Subject: PR/48489 CVS commit: src/bin/sh
Date: Wed, 1 Jun 2016 02:47:06 +0000

 Module Name:	src
 Committed By:	kre
 Date:		Wed Jun  1 02:47:05 UTC 2016
 
 Modified Files:
 	src/bin/sh: parser.c
 
 Log Message:
 PR bin/51145 PR bin/48489
 More fixes to the shell parser to prevent empty simple commands (where
 empty means no significant text at all) - as discussed on tech-userlevel
 this still allows { } (which can be useful in function definitions, not
 really anywhere else though) except in posix mode.  ( ) now generates
 a syntax error, as should any other place where commands are required but
 nothing is present.  (nb, redirections, var assignments, even var expansions
 that expand to nothing, are all OK, and avoid the error - just comments, or
 whits space, are not.)    This is (aside from allowing { } at all) all in
 accordance with the posix spec.
 
 
 To generate a diff of this commit:
 cvs rdiff -u -r1.119 -r1.120 src/bin/sh/parser.c
 
 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.
 


Home | Main Index | Thread Index | Old Index