httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "William A. Rowe, Jr." <wr...@rowe-clan.net>
Subject Re: ApacheMonitor 1.1
Date Tue, 14 Aug 2001 19:37:25 GMT
From: "Mladen Turk" <mladen.turk@mail.inet.hr>
Sent: Sunday, August 12, 2001 1:39 PM


> CHANGES:
> 1. Cleaned the ApacheMonitor code a little bit.
> 2. Removed two icons apmall.ico and apsrvmon.ico
> 3. Added option to recognize the Apache*.exe (e.g. Apache-1.3.20.exe)
> 4. Removed AboutBox
> 5. Changed refresh time to 2/20 sec.

Mladen,

  now that this is part of the Apache project, we need to ask you to do two
major things (I committed, this time, in spite of these);

  submit diffs (unless you are adding an entirely new file.)

  focus each patch on doing one thing.  The symbol renames from ap_ to g_,
for example, could have been a single patch into and of itself, so nobody gets
confused figuring out what the difference is.

  Others work with the code in CVS, so it's important that everyone can understand
the changes as they happen.  Also, it makes other's patches less 'trounced' on a
given update.

  I have a couple of patches near-ready to APR, to expose some things that the
ApacheMonitor and Win32 MPM need to see.  Once those are committed, I'll be able
to trim some code from the monitor.  I'll also be looking at using the left/right
click to list or control the services (as opposed to pulling up the entire dialog.)
Working -with- cvs makes it simpler for everyone.

  I'm beginning to think that a 'Daemon Enumerator' function would be pretty slick
in apr, that could use the service or process semantics on a given platform. 
Not something I'm up to attacking this month, though :-)

Bill



Mime
View raw message