Re: docs/87351: Jail building instructions don't work as described

Call me madcap, but wouldn't it make more sense to have the base case 
example match the default shell, with exceptions for other shells as the 
side-comments?

Given how little of this script/instructions need to change to suit csh, 
something like...

[...]
#For csh
	set D=/here/is/the/jail
#For bourne shells (sh,bash,...)
	D=/here/is/the/jail
cd /usr/src
mkdir -p $D
[...]

or

[...]
#For csh
	set D=/here/is/the/jail
#For bourne shells (sh,bash,...)
#	D=/here/is/the/jail
cd /usr/src
mkdir -p $D
[...]

The former solution is both legible and functional. If you dump it on 
the command-line verbatim, it will produce an error for whichever 
environment export didn't work, but the process overall will work, and 
the latter is a little cleaner, and won't produce any errors with the 
base shell. A happy user experience overall.

-Eli


Giorgos Keramidas wrote:
> Synopsis: Jail building instructions don't work as described with default CSH shell
> 
> State-Changed-From-To: open->patched
> State-Changed-By: keramida
> State-Changed-When: Wed Oct 26 20:18:41 GMT 2005
> State-Changed-Why: 
> 
> I've just committed a note that points to sh(1) in CURRENT.
> The change will be MFC'ed in a few days.  If I get approval
> early enough it will even be in 6.0-RELEASE.
> 
> Thanks for the problem report :)
> 
> http://www.freebsd.org/cgi/query-pr.cgi?pr=87351
_______________________________________________
[e-mail] mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-doc
To unsubscribe, send any mail to "[e-mail]"