Actually looking at my patch I'm now rethinking my last statement... Whit that patch I have now firstidx used by scmbug for any Bugzilla version (not just for 4.0 and higher). This should work fine but means dependency should be added to scmbug too (with this approach static module loading should be fine).
Alternative scenario would be using Bugzilla::Util::lsearch for old Bugzilla versions and firstidx for newer ones but 1) this would make code look a bit more "ugly"; 2) as I see in other discussions in the mailing list - dynamic module loading seems to cause some issues - Yavor On Sun, Mar 6, 2011 at 16:33, Kristis Makris <[email protected]> wrote: > On Sat, 2011-03-05 at 17:47 +0200, Yavor Nikolov wrote: > > I'm just not sure whether it wouldn't be better to have that "use > > List::MoreUtils" dynamically evaluated since: - it's specific for > > Bugzilla 4.0 and higher - it could reside in $BUGZILLA_ROOT/lib only > > You are right. It should be dynamically evaluated. > > I am not sure it could only reside in $BUGZILLA_ROOT/lib though. It's > just another Perl module. >
_______________________________________________ scmbug-users mailing list [email protected] http://lists.mkgnu.net/cgi-bin/mailman/listinfo/scmbug-users
