Paul Johnson
2012-08-01 15:43:31 UTC
In accordance with the terms of my grant from TPF this is the report for
week 9 of my work on improving Devel::Cover.
This report covers 14.07 to 20.07.
This week I released Devel::Cover 0.91 and 0.92.
I made a change I've been considering for quite some time, which is to
get the @INC directories from perl itself.
At build time we put the @INC directories in @Devel::Cover::Inc::Inc so
that we will be able to ignore them by default at runtime. That works
well normally, but sometimes the build environment differs from the run
environment which can result in default coverage of core modules.
The solution is to shell out to perl and ask it what its @INC is. Of
course it's not quite that easy.
And shelling out isn't very nice, but when the new V module for
introspection is in place we can use that instead.
I knew this was a risky commit, but cpantesters didn't show any
problems. Nevertheless, I had broken things for people using
local::lib. Thankfully, Olivier Mengué noticed and patched it up for
me.
I also fixed up a change I made last week which made all the tests fail
on Windows. Devel::Cover itself still worked, but almost all of the
tests didn't. Christian Walde's cpantesters smoker detected the problem
quickly and we diagnosed the problem together on IRC. So I eventually
got around to installing Perl on Windows (which turned out to be pretty
simple) and then it was just a matter of getting the quoting such that
both *nix and Windows were happy, and hoping that other platforms work
too.
(On which note, does anyone use Devel::Cover on anything other than *nix
and Windows?)
As usual, I made other various fixes and updates.
Closed RT tickets:
68991 t/e2e epic fail when prefix for perl is not the physical location
76528 Devel/Cover/Inc.pm contains temporary install destination DESTDIR
66881 Symlinked directories can cause ignore functionality to fail
37349 /usr/local/* ommitted in Inc
32345 Devel::Cover 0.63 fails 1 test under Perl 5.8.7, Ubuntu 6.06
Closed Github tickets:
25 Fix inc directories when working with local::lib
Merged pull requests:
25 Fix inc directories when working with local::lib
Fixed cpantesters reports:
http://www.cpantesters.org/cpan/report/251a6589-6ef1-1014-92c1-140f3373d4b5
http://www.cpantesters.org/cpan/report/d50bb259-6bf5-1014-86c6-f8fbafede0e8
http://www.cpantesters.org/cpan/report/675f887f-6fa2-1014-9b4e-081b3373d4b5
You can see the commits at https://github.com/pjcj/Devel--Cover/commits/master
Hours worked:
14.07 03:20
16.07 04:25
20.07 01:15
Total 09:00
week 9 of my work on improving Devel::Cover.
This report covers 14.07 to 20.07.
This week I released Devel::Cover 0.91 and 0.92.
I made a change I've been considering for quite some time, which is to
get the @INC directories from perl itself.
At build time we put the @INC directories in @Devel::Cover::Inc::Inc so
that we will be able to ignore them by default at runtime. That works
well normally, but sometimes the build environment differs from the run
environment which can result in default coverage of core modules.
The solution is to shell out to perl and ask it what its @INC is. Of
course it's not quite that easy.
And shelling out isn't very nice, but when the new V module for
introspection is in place we can use that instead.
I knew this was a risky commit, but cpantesters didn't show any
problems. Nevertheless, I had broken things for people using
local::lib. Thankfully, Olivier Mengué noticed and patched it up for
me.
I also fixed up a change I made last week which made all the tests fail
on Windows. Devel::Cover itself still worked, but almost all of the
tests didn't. Christian Walde's cpantesters smoker detected the problem
quickly and we diagnosed the problem together on IRC. So I eventually
got around to installing Perl on Windows (which turned out to be pretty
simple) and then it was just a matter of getting the quoting such that
both *nix and Windows were happy, and hoping that other platforms work
too.
(On which note, does anyone use Devel::Cover on anything other than *nix
and Windows?)
As usual, I made other various fixes and updates.
Closed RT tickets:
68991 t/e2e epic fail when prefix for perl is not the physical location
76528 Devel/Cover/Inc.pm contains temporary install destination DESTDIR
66881 Symlinked directories can cause ignore functionality to fail
37349 /usr/local/* ommitted in Inc
32345 Devel::Cover 0.63 fails 1 test under Perl 5.8.7, Ubuntu 6.06
Closed Github tickets:
25 Fix inc directories when working with local::lib
Merged pull requests:
25 Fix inc directories when working with local::lib
Fixed cpantesters reports:
http://www.cpantesters.org/cpan/report/251a6589-6ef1-1014-92c1-140f3373d4b5
http://www.cpantesters.org/cpan/report/d50bb259-6bf5-1014-86c6-f8fbafede0e8
http://www.cpantesters.org/cpan/report/675f887f-6fa2-1014-9b4e-081b3373d4b5
You can see the commits at https://github.com/pjcj/Devel--Cover/commits/master
Hours worked:
14.07 03:20
16.07 04:25
20.07 01:15
Total 09:00
--
Paul Johnson - ***@pjcj.net
http://www.pjcj.net
Paul Johnson - ***@pjcj.net
http://www.pjcj.net