While fixing bug 585028, I applied the patch to all branches, assuming it was fine to use List::MoreUtils in all cases. But checksetup.pl only requires this module in Bugzilla 4.0 and newer, not in 3.6. This means that those who don't have this module installed now get an error everytime Bugzilla::Search::Quicksearch is called, when upgrading from 3.6.2 to 3.6.3. I needed this module to use the 'firstidx' subroutine. For 3.6.x, we have to use something else.
Oh, yeah, use lsearch for 3.6.x.
Assignee: query-and-buglist → LpSolit
Status: NEW → ASSIGNED
Attachment #489646 - Flags: review?(mkanat)
Comment on attachment 489646 [details] [diff] [review] patch for 3.6.3, v1 Looks good to me. :-)
Attachment #489646 - Flags: review?(mkanat) → review+
Committing to: bzr+ssh://email@example.com/bugzilla/3.6/ modified Bugzilla/Search/Quicksearch.pm Committed revision 7206.
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.