On Jul 14, 2007, at 1:52 AM, Knut Urdalen wrote:

Hi all,

I've just submitted the initial code for the PHP 5 port [1].

svn checkout http://svn.apache.org/repos/asf/incubator/log4php


The code can currently be found in src/php5/*

This revision contains:
- working code for most appenders
- an examples folder [2] with working examples (go to the folder and run any PHP file with the CLI) - some unit tests using PHPUnit 3 (go to src/php5/tests and run "phpunit AllTests.php" add "--report=coverage" if you have xdebug installed to get a nice code coverage report) - a TODO-list [3] with a lot of tasks to choose from (feel free to grab one or more tasks - just inform us here on the mailing list and we'll help you out)

This is our starting point for the PHP 5 port. Please start testing this version in your environment to see if it works as expected.

Any feedback is appreciated!

Best regards,
Knut Urdalen

[1] http://svn.apache.org/viewvc?view=rev&revision=556231
[2] http://svn.apache.org/viewvc/incubator/log4php/trunk/src/php5/ examples/ [3] http://svn.apache.org/viewvc/incubator/log4php/trunk/src/php5/ TODO?view=markup

I do not know why log4php was still showing up in the https://
svn.apache.org/repos/asf/logging.  I thought I had svn mv'd it, but
if I inadvertently svn cp'd it, I've not svn rm'd the original in rev
556719.

Subversion will generate a commit email message on changes and send
it to log4php-dev.  Unfortunately, since Knut was not subscribed to
log4php-dev as [EMAIL PROTECTED], the message was rejected as
spam.  I've forcibly subscribed the log4php committers to log4php-dev-
allow (a list consulted to determine if messages should be accept
using the @apache.org addresses), so hopefully any subsequent commit
messages will get through even if you are not subscribed to log4php-
dev using your @apache.org address.

Rev 556231 lost the revision history since files were file system
copied from src/log4php to src/php5/log4php and then committed as
brand new files apparently.  It would
have been preferable to svn cp or svn mv the PHP 4 files and then
apply the PHP 5 changes.  I've going to shortly commit a rebuilt
log4php source code in src/main/php by moving src/log4php (the
current PHP 4 source code) and then apply a patch that updates that
source to be identical (other than whitespace) to src/php5/log4php.
There were a lot of whitespace changes which I assume are
unnecessary.  If the whitespace changes are necessary, then it would
be better to commit the non-whitespace changes and then run a code
formatter on the whole lot.

@author tags are discouraged in ASF code (http://www.apache.org/
foundation/records/minutes/2004/board_minutes_2004_09_22.txt, perhaps
there is a better reference), but there hasn't been any effort to
remove them in LS code.  I did see that Knut added author tags, maybe
it would be better to start removing the ones that are there instead
of adding new ones.


Reply via email to