Hi Spoorthy,

Spoorthy H.S said the following on Monday 30 March 2009 02:56 PM:
> Yeah. Changed the line in prototype_com file.
> Here's the new webrev.
> http://cr.opensolaris.org/~spoorthy/conmanupd5/
>
> Thanks
> Spoorthy
Below are few comments:

1. In usr/src/cmd/conman/conmand.xml, you are exec'ing the svc method 
file. So it will try to start the conman daemon instead of stopping it.
>   41         <exec_method
>   42             type='method'
>   43             name='stop'
>   44             exec='/lib/svc/method/svc-conmand'
>   45             timeout_seconds='60' />

So you may want to kill/stop the current daemon process. Something like 
below:

      41         <exec_method
      42             type='method'
      43             name='stop'
      44             *exec=':kill'*
      45             timeout_seconds='60' />



2. In install-sfw and SUNWconmanu/prototype_com files, there is 
inconsistency with the permissions for README file. Please change the 
permission for README file to 0444 at all places.

Rest of the changes looks fine.

Thanks,
Srirama
-------------- next part --------------
An HTML attachment was scrubbed...
URL: 
<http://mail.opensolaris.org/pipermail/sfwnv-discuss/attachments/20090401/a0022aba/attachment.html>

Reply via email to