On Fri, Sep 25, 2015 at 12:24 +0200, Ronny Pfannschmidt wrote: > Before we can merge it we need to restructure capturing, since logcapture > currently incurs a huge perf cost (at least it did the last time we tried to > merge it)
can't remember but ok :) On a cursory look it seems that capturelog creates and configures a LogHandler for each test. We can isntead probably just create one log handler per session and snapshot relevant loglines per test. Also we probably want to add an option "--log-capture=yes" (default no or "auto"). Lastly, i think we should consider merging capturelog into the core capture.py plugin as it's very related. Maybe I can give a PR a try next week. In any case, pytest-2.9 is not going to be out soon so we'd have some time to play with and settle mechanics. holger > Am 25. September 2015 11:43:16 MESZ, schrieb holger krekel > <[email protected]>: > >Hi all, > > > >pytest-capturelog has been abandoned [1] but is still getting used a > >lot > >as far as i see. As it basically helps dealing with capturing logging > >and logging is part of the python standard lib i think we could see > >to merge it into pytest and maintain it from there. Is anyone up > >for trying to do this along the lines of pytest-cache? > > > >FWIW I expect less breakage with pytest-capturelog because there is > >no change like there was with the execnet->json move while merging. > >It should be a relatively straightforward task. > > > >Alternatively we could see to get maintainership from pypi admins > >but i think in this case it's not worth it. > > > >best, > >hogler > > > > > >[1] I and others tried to contact the author several times to no avail. > > > >_______________________________________________ > >pytest-dev mailing list > >[email protected] > >https://mail.python.org/mailman/listinfo/pytest-dev > > MFG Ronny > -- about me: http://holgerkrekel.net/about-me/ contracting: http://merlinux.eu _______________________________________________ pytest-dev mailing list [email protected] https://mail.python.org/mailman/listinfo/pytest-dev
