On Fri, 19 Feb 2010 15:39:21 +0100, Jerry Jelinek <gerald.jeli...@sun.com>
> 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
> 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:
PAGER /usr/bin/nonsense does not exist (No such file or directory).
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.
zones-discuss mailing list