On Sat, Apr 6, 2013 at 8:43 PM, Peter Rabbitson <notifications@github.com>wrote:<br>
<br>
> The correct fix (loading DBI.pm is *not* it, see commit msg above) is<br>
> already contained in the 0.08250-to-be branch (<br>
> https://github.com/dbsrgits/dbix-class/tree/topic/constructor_rewrite).<br>
><br>
I figured the fix would be more involved than that, that's why I didn't<br>
include any try of my own to fix it.<br>
<br>
> Please do not adjust the HFH::M::DBIC tests - they are valid.Could you<br>
> also test HFH against ac09a39<https://github.com/dbsrgits/dbix-class/commit/ac09a390ca>to make sure everything works as intended?<br>
><br>
After cherry-picking my test into this commit no more errors are thrown.<br>
The test does exactly what HFH::m::DBIC does (it's modeled after the<br>
failure there, after all) so everything seems to work fine. If you need<br>
further testing I might be able to to that tomorrow.<br>
<br>
Cheers,<br>
Sebastian
<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br>Reply to this email directly or <a href='https://github.com/dbsrgits/dbix-class/pull/23#issuecomment-16002086'>view it on GitHub</a>.<img src='https://github.com/notifications/beacon/4W3BWTo7EIi_7vx28Xzl_ibG_jBay88FdWCJyhoDCHKbZ747zHJIHKUQjIL36Oj1.gif' height='1' width='1'></p>