On 02/19/10 10:32 AM, Frank Batschulat (Home) wrote:
On Fri, 19 Feb 2010 15:39:21 +0100, Jerry Jelinek<gerald.jeli...@sun.com>
wrote:
On 02/19/10 06:53, Frank Batschulat (Home) wrote:
May I request 2 code reviewers for the changes for:
6914152 zonecfg fails when less(1M) is missing
http://bugs.opensolaris.org/bugdatabase/view_bug.do?bug_id=6914152
http://cr.opensolaris.org/~batschul/zpager/
This looks fine to me. One nit:
911& 5192 The error says ""Could not stat PAGER". This error
message might be useful to a developer
but isn't that useful for a sysadmin. Can you print
something more meaningful like "PAGER %s does not exist"
Thanks Jerry, that is indeed a valid concern, I changed it to be:
<snip>
PAGER /usr/bin/nonsense does not exist (No such file or directory).
<snip end>
I included the real error string in case of permission errors where the
file does indeed exist and I am now dropping the mysterious "stat" part.
updated webrev:
http://cr.opensolaris.org/~batschul/zpager/
cheers
frankB
_______________________________________________
zones-discuss mailing list
zones-discuss@opensolaris.org
Hi Frank,
This looks fine to me.
Jordan
_______________________________________________
zones-discuss mailing list
zones-discuss@opensolaris.org