|
From: Kevin G. <ke...@go...> - 2002-09-24 14:47:26
|
1) First, nice work Mike on making Time::HiRes optional, I just noticed
that.
2) I fired up Log4perl on Win32 perl 5.6.1 (Activestate). Here are all
the modules I had to add, if anyone's curious what our base dependencies
are:
Attribute-Handlers
Params-Validate
Log-Dispatch
Test-Simple
Test-Harness
3) t/010JConsole.t used a call to `touch`, which wasn't causing an error
but I fixed anyway.
4) These guys were looking for '/' as a directory separator in their
test results, I changed the ok() comparisons to use regexes with a '.'
instead, that'll match unix '/' and dos '\', but I think a mac is '::'.
There doesn't seem to be an easy way to get the directory separator on
the current machine, anybody know one?
003Layout.t
005Config-Perl.t
006Config-Java.t
007LogPrio.t
(changes checking in now)
5) 020Easy.t is using a unixism to capture STDERR, we need a different
way to test that, but I'm out of time right now
# Test 3 got: '' (t\020Easy.t at line 37)
# Expected: 'm#this we want#'
not ok 4
# Test 4 got: '' (t\020Easy.t at line 38)
# Expected: 'm#this also#'
FAILED tests 3-4
Failed 2/4 tests, 50.00% okay
6) 022Wrap.t I think is showing the %F{1} doesn't work with the windows
'\', but I don't have time to take it further right now.
# Test 1 got: 'File: t\022Wrap.t Line number: 60 package: main'
(t\022Wrap.t at
line 62)
# Expected: 'File: 022Wrap.t Line number: 60 package: main'
FAILED test 1
msc...@ao... wrote:
>
> In a message dated Mon, 23 Sep 2002 7:10:44 PM Eastern Standard Time, ke...@go... writes:
>
> >I just built a 5.8 (not 5.6.1) with threads and Log4perl tested fine,
> >I'll do more investigation and let you know.
>
> That's good to know -- my suspicion is, though, that it's more Windows- than thread-related.
>
> -- Mike
>
> ############################
> # Mike Schilli #
> # log...@pe... #
> # http://perlmeister.com #
> # log4perl.sourceforge.net #
> ############################
>
> -------------------------------------------------------
> This sf.net email is sponsored by:ThinkGeek
> Welcome to geek heaven.
> http://thinkgeek.com/sf
> _______________________________________________
> log4perl-devel mailing list
> log...@li...
> https://lists.sourceforge.net/lists/listinfo/log4perl-devel
--
Happy Trails . . .
Kevin M. Goess
(and Anne and Frank)
904 Carmel Ave.
Albany, CA 94706
(510) 525-5217
|