| Subject: | Check availability of compact function before trying to compact. |
| Date: | Fri, 8 Jul 2011 15:44:55 +0200 |
| To: | <bug-Cache-BDB [...] rt.cpan.org> |
| From: | Kevin Frost <Kevin_Frost [...] symantec.com> |
This one might be a real code bug, or it may be that I'm missing something.
As noted in my previous report, t/02-api.t checks whether "compact" is supported and skips some tests if it's not.
However, in lib/Cache/DBD.pm there is no equivalent check. This results in an error:
"Your vendor has not defined BerkeleyDB macro DB_FREE_SPACE"
I think it should rather default to disable_compact if compact is not supported according to the logic in t/02-api.t. Patch attached (including the test count change) if you want it.
In case I'm misinterpreting the situation and the problem is on my end, my end is:
RHEL5, perl 5.12.1 compiled from source; Berkeley DB is:
db4-4.3.29-10.el5
db4-devel-4.3.29-10.el5
cheers
-- f.
This one might be a real code bug, or it may be that I'm missing something.
As noted in my previous report, t/02-api.t checks whether "compact" is supported and skips some tests if it's not.
However, in lib/Cache/DBD.pm there is no equivalent check. This results in an error:
"Your vendor has not defined BerkeleyDB macro DB_FREE_SPACE"
I think it should rather default to disable_compact if compact is not supported according to the logic in t/02-api.t. Patch attached (including the test count change) if you want it.
In case I'm misinterpreting the situation and the problem is on my end, my end is:
RHEL5, perl 5.12.1 compiled from source; Berkeley DB is:
db4-4.3.29-10.el5
db4-devel-4.3.29-10.el5
cheers
-- f.
Message body is not shown because sender requested not to inline it.