Re: [Call for Votes] System_ProcWatch

bit of a bikeshed comment, but abstracting the configuration from the class looks like a good idea: $xml_us = &new XML_Unserializer(); $xml_us->unserialize('somexml.xml', true); $config = $xml_us->getUnserializedData(); // or $config = array('......'=>.....) ????? $procwatch = new System_ProcWatch($config); $procwatch->run($ps_args); Unless I'm missing something, that would enable you to embed it into other applications alot easier., and just use arrays or whatever to configure it.. Regards Alan Arnaud Limbourg wrote:

cursory review +1 a very few minor cs issues foreach (){ instead of foreach () { but i'm just being picky. Arnaud. Michael Wallner wrote:

It may be a bad time, but this project has matured nicely in the meantime, so I'd like to call for votes on System_ProcWatch. As this package doesn't fit well into other categories it is proposed for a new category "System", if there is general interest in having this package in PEAR. If you haven't already been there, have a look at: http://dev.iworks.at/PEAR/proposals/System_ProcWatch.html or try it yourself: http://dev.iworks.at/PEAR/System_ProcWatch-0.1.tgz I would appreciate some reports from other *nix systems than Linux :) The package comes with a shell script for direct use. An example configuration file is also provided, which is ready for use at [PEAR_DIR]/docs/System_ProcWatch/example.xml The procwatch.dtd should help you starting to write your own config files. Regards,

-- Can you help out? Need Consulting Services or Know of a Job? http://www.akbkhome.com