Since this "did" pass tests on the Windows platform, does that mean I only owe half-a-beer to everyone?
-Nathan > -----Original Message----- > From: Geir Magnusson Jr [mailto:[EMAIL PROTECTED] > Sent: Monday, July 17, 2006 12:29 PM > To: harmony-dev@incubator.apache.org > Subject: [classlib][logging] Test Failures - FileHandlerTest (was Re: > [bild] linux failure) > > Ok, the problem is that we're including FileHandlerTest, there are a few > bugs (maybe) and our implementation differs from the RI, and the failure > is platform dependent to boot. > > First bug, in FileHandlerTest.testInvalidParams() we have things like : > > // %t and %p parsing can add file separate automatically > FileHandler hl = new FileHandler("%taaa"); > > First problem is that our implemention *doesn't* add the separator, so > the result is that logging is trying to create (on linux) > > /tmpaaa > > and given that my root dir is locked down (and I don't run as root!), > creating that file failed. On my windows box, this passed because > java.io.tmpdir ends w/ a separator, so it creates the thing as expected. > > Now, the RI on linux puts in the separator, so the result is > > /tmp/aaa > > Reading the spec, this is not what I expected, as there is a separator > "/" defined, and all examples use it, and nothing is said about it. > > So, we can either fix the test, or given that the RI does behave beyond > the spec (and adds the separator), I suppose that we should just simply > add a separator in the implementation for %t and %h? > > geir > > > Geir Magnusson Jr wrote: > > Ok, I understand why. I'll either fix, or post an item for discussion > > re the fix. > > > > I think Nathan owes us a beer. > > > > geir > > > > > > Tim Ellison wrote: > >> Sorry, yes I meant the test run. > >> > >> Regards, > >> Tim > >> > >> Geir Magnusson Jr wrote: > >>> Oh, I can build, but tests don't pass. > >>> > >>> geir > >>> > >>> Vladimir Gorr wrote: > >>>> I was able to successfully build on Linux for the recent sources. > >>>> > >>>> Thanks, > >>>> Vladimir. > >>>> > >>>> > >>>> > >>>> On 7/17/06, Tim Ellison <[EMAIL PROTECTED]> wrote: > >>>>> The linux build is failing as follows, I have not investigated: > >>>>> > >>>>> java.io.FileNotFoundException at > >>>>> > org.apache.harmony.luni.platform.OSFileSystem.open(OSFileSystem.java:223) > >>>>> at java.io.FileOutputStream.<init>(FileOutputStream.java:92) at > >>>>> java.io.FileOutputStream.<init>(FileOutputStream.java:155) at > >>>>> java.util.logging.FileHandler.initOutputFiles(FileHandler.java:207) > at > >>>>> java.util.logging.FileHandler.init(FileHandler.java:190) at > >>>>> java.util.logging.FileHandler.<init>(FileHandler.java:386) at > >>>>> > >>>>> > org.apache.harmony.logging.tests.java.util.logging.FileHandlerTest.testInv > alidParams > >>>>> > >>>>> (FileHandlerTest.java:453) > >>>>> at > java.lang.reflect.AccessibleObject.invokeV(AccessibleObject.java:205) > >>>>> > >>>>> > >>>>> -- > >>>>> > >>>>> Tim Ellison ([EMAIL PROTECTED]) > >>>>> IBM Java technology centre, UK. > >>>>> > >>>>> -------------------------------------------------------------------- > - > >>>>> Terms of use : http://incubator.apache.org/harmony/mailing.html > >>>>> To unsubscribe, e-mail: [EMAIL PROTECTED] > >>>>> For additional commands, e-mail: harmony-dev- > [EMAIL PROTECTED] > >>>>> > >>>>> > >>> --------------------------------------------------------------------- > >>> Terms of use : http://incubator.apache.org/harmony/mailing.html > >>> To unsubscribe, e-mail: [EMAIL PROTECTED] > >>> For additional commands, e-mail: [EMAIL PROTECTED] > >>> > >>> > > > > --------------------------------------------------------------------- > > Terms of use : http://incubator.apache.org/harmony/mailing.html > > To unsubscribe, e-mail: [EMAIL PROTECTED] > > For additional commands, e-mail: [EMAIL PROTECTED] > > > > > > > > --------------------------------------------------------------------- > Terms of use : http://incubator.apache.org/harmony/mailing.html > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] --------------------------------------------------------------------- Terms of use : http://incubator.apache.org/harmony/mailing.html To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
