Hi Massimo Well, who would have thought: I made the testsuite work for the worker mpm, by just giving my pseudo kill proc to fake TclX a bit more time (5 secs) to do the job ☺. I noticed that I didn’t get the same result always, that’s why I suspected a timing issue… The tests run forever now, but nothing fails for the worker mpm anymore!
I still got only 83 tests though, but I’ll have to look into that another time. As for now, I guess, I scared you for nothing. Sorry for that! Gruess Ronnie -- We innovate. You win. http://netcetera.com Ronnie Brunner | ronnie.brun...@netcetera.com | T +41 44 297 59 79 | Netcetera AG | 8040 Zürich | Switzerland | http://netcetera.com | From: Massimo MANGHI [mailto:massimo.man...@unipr.it] Sent: Dienstag, 6. Februar 2018 21:01 To: Ronnie Brunner <ronnie.brun...@netcetera.com>; Massimo Manghi <massimo.man...@alice.it>; petas...@yahoo.gr Cc: rivet-dev@tcl.apache.org Subject: Re: Rivet 3.0.1 Thank you Ronnie, still the discrepancy between my PC and yours bothers me quite a bit. I would like to have more time to investigate, in the meanwhile would you sent over your error_log and output? Thank you -- Massimo Manghi ________________________________ Da: Ronnie Brunner <ronnie.brun...@netcetera.com<mailto:ronnie.brun...@netcetera.com>> Inviato: martedì 6 febbraio 2018 20:48 A: Massimo MANGHI; Massimo Manghi; petas...@yahoo.gr<mailto:petas...@yahoo.gr> Cc: rivet-dev@tcl.apache.org<mailto:rivet-dev@tcl.apache.org> Oggetto: RE: Rivet 3.0.1 Hi Massimo Sorry, I haven’t noticed your answer w/ the rc2 earlier. Thought it referred to George’s issue with the test suite… I have compared 3.0.1rc1 w/ 3.0.1rc2: they show the same result. Running tests now gives 38 failed tests for both. (I’m not sure why I got fewer errors when I tried initially. Could well be that some of it is and or was timing related, the machine I use is used by many and although virtualized, it’s not always under the same or even just similar load…) So I stand by the statement: it’s an improvement since 3.0.0, so nothing speaks against a release, but there is still something amiss. (No guarantee it’s not my somewhat homegrown environment: I compiled httpd as well as Tcl locally to have control over the versions and I refer to only those versions when testing. Great for testing different versions, but as I learned, the test suite currently runs on Tcl8.6 and http 2.4 only…. I use Tcl.8.6.6 and hddpt-2.4.27 currently) Regards Ronnie -- We innovate. You win. http://netcetera.com Ronnie Brunner | ronnie.brun...@netcetera.com<mailto:ronnie.brun...@netcetera.com> | T +41 44 297 59 79 | Netcetera AG | 8040 Zürich | Switzerland | http://netcetera.com | From: Massimo MANGHI [mailto:massimo.man...@unipr.it] Sent: Dienstag, 6. Februar 2018 18:33 To: Ronnie Brunner <ronnie.brun...@netcetera.com<mailto:ronnie.brun...@netcetera.com>>; Massimo Manghi <massimo.man...@alice.it<mailto:massimo.man...@alice.it>>; petas...@yahoo.gr<mailto:petas...@yahoo.gr> Cc: rivet-dev@tcl.apache.org<mailto:rivet-dev@tcl.apache.org> Subject: Re: Rivet 3.0.1 Hi Ronnie are your latest test results obtained from 3.0.1rc2? I fixed a problem that made the server fail in most cases, their are all OK for me now Another issue with your test output is the overall number of tests actually carried out: 83. I have 130 tests running and 3.0.1rc2 passes each and everyone on my PC. Why your test suite is avoiding ~50 tests? -- Massimo ________________________________ Da: Ronnie Brunner <ronnie.brun...@netcetera.com<mailto:ronnie.brun...@netcetera.com>> Inviato: martedì 6 febbraio 2018 17:05 A: Massimo MANGHI; Massimo Manghi; petas...@yahoo.gr<mailto:petas...@yahoo.gr> Cc: rivet-dev@tcl.apache.org<mailto:rivet-dev@tcl.apache.org> Oggetto: RE: Rivet 3.0.1 Hi Massimo My problem for the worker mpm is certainly not a blocker, as I never managed (actually, really tried) to run the tests for that mpm. I always tested exclusively against prefork, mainly because I thought the others aren’t supported yet. Also: The test result of Rivet 3.0.0 that I get in my setup is much worse than the result of 3.0.1. So 3.0.1 actually brings an improvement as previously, almost no tests worked. (I just reran my setup w/ 3.0.0 -> rivet.test: Total 83 Passed 4 Skipped 0 Failed 79) Hth and best regards Ronnie -- We innovate. You win. http://netcetera.com Ronnie Brunner | ronnie.brun...@netcetera.com<mailto:ronnie.brun...@netcetera.com> | T +41 44 297 59 79 | Netcetera AG | 8040 Zürich | Switzerland | http://netcetera.com | From: Massimo MANGHI [mailto:massimo.man...@unipr.it] Sent: Dienstag, 6. Februar 2018 16:29 To: Ronnie Brunner <ronnie.brun...@netcetera.com<mailto:ronnie.brun...@netcetera.com>>; Massimo Manghi <massimo.man...@alice.it<mailto:massimo.man...@alice.it>>; petas...@yahoo.gr<mailto:petas...@yahoo.gr> Cc: rivet-dev@tcl.apache.org<mailto:rivet-dev@tcl.apache.org> Subject: Re: Rivet 3.0.1 Should we consider this issue a reason for withholding 3.0.1? I think that as long as 2.2 is supported we may exploit the compatibility with 2.4. When 2.2 is deprecated we may proceed with the new macro definition. This is good example where having a versioning system like git or fossil could be handy. Can I start the vote for releasing 3.0.1? -- Massimo ________________________________ Da: Georgios Petasis <petas...@yahoo.gr<mailto:petas...@yahoo.gr>> Inviato: domenica 4 febbraio 2018 14:54 A: Massimo MANGHI; Ronnie Brunner; Massimo Manghi Cc: rivet-dev@tcl.apache.org<mailto:rivet-dev@tcl.apache.org> Oggetto: Re: Rivet 3.0.1 Dear Massimo, According to the documentation: https://httpd.apache.org/docs/2.4/developer/new_api_2_4.html We should also modify rivet.c to use AP_DECLARE_MODULE instead of "module AP_MODULE_DECLARE_DATA rivet_module = " we now have. I have tried it, and there was no difference (of course we must take care the old way is used in versions < 2.4 of apache). I tried to run the tests, but I am missing the TclX package. Where can I find its sources? In sourceforge? Best, George Στις 3/2/2018 20:52, ο Massimo Manghi έγραψε: > I've just uploaded to > www.rivetweb.org/~mxm/rivet<http://www.rivetweb.org/~mxm/rivet> a RC2 tar > archive > with a proposed patch of the problem that made most tests fail. > > The bug has surfaced when George introduced the APLOG_USE_MODULE in > mod_rivet.h, since also rivetlib/rivetPkgInit.c used it in order to > get the definition of the module globals. But since the generation of > the pkgIndex.tcl file takes place outside of mod_rivet the mk_pkgIndex > failed for rivetlib and became impossible to mod_rivet to find the > library. As as consequence the server failed because rivetlib is > loaded in the process of initializing a Rivet interpreter > > Actually the introduction of the macro was correct and it's was > librivet/rivetPkgInit.c duty to determine the ::rivet name space > pointer in a different and sane way. This is the crux of the patch > included in RC2 > > -- Massimo > > On 02/01/2018 01:07 PM, Ronnie Brunner wrote: >> Hi Massimo >> >> Attached the output (stout and sdterr) of running the test suite >> (including the compilation of rivet), the error_log and access_log >> from rivet-3.0.1/tests after running the test suite. >> >> I'm running a CentOS Linux release 7.1.1503 (Core) os for these >> tests, Tcl8.6.6 and httpd 2.4.27 with the worker mpm. The only >> modification to runtests.tcl was to remove the prefork module check, >> which obviously fails if you run the worker mpm. >> >> Let me know if you need anything else. >> >> Hth Ronnie >> > > --------------------------------------------------------------------- > To unsubscribe, e-mail: > rivet-dev-unsubscr...@tcl.apache.org<mailto:rivet-dev-unsubscr...@tcl.apache.org> > For additional commands, e-mail: > rivet-dev-h...@tcl.apache.org<mailto:rivet-dev-h...@tcl.apache.org> > >