Summary: | Add Mojolicious as a required dependency | ||
---|---|---|---|
Product: | Koha | Reporter: | David Cook <dcook> |
Component: | Architecture, internals, and plumbing | Assignee: | David Cook <dcook> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | trivial | ||
Priority: | P5 - low | CC: | mtj, tomascohen |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | Trivial patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Circulation function: | |||
Bug Depends on: | 17932 | ||
Bug Blocks: | |||
Attachments: | Bug 18326 - Add Mojolicious as a required dependency |
Description
David Cook
2017-03-24 01:15:18 UTC
Created attachment 61593 [details] [review] Bug 18326 - Add Mojolicious as a required dependency Since Bug 17932 was pushed and adds Mojo::JSON to Koha::Object, it's necessary to specify Mojolicious as a required module. _TEST PLAN_ 1. Run: perl koha_perl_deps.pl -a -r -c 2. Make sure that Mojolicious is in the output Maybe bug 18290 could provide an alternative solution. (In reply to Tomás Cohen Arazi from comment #2) > Maybe bug 18290 could provide an alternative solution. Sounds good to me. I've already installed Mojolicious so that I can continue to use Koha master, but an alternative solution seems reasonable! I think this patch might be obsolete - it's already marked required on master: }, 'Mojolicious' => { 'usage' => 'REST API', 'required' => '1', 'min_ver' => '6.0', }, Not sure which status... choosing to close, please reopen if I am mistaken. |