| 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 | ||
| GIT URL: | Initiative type: | --- | |
| Sponsorship status: | --- | Comma delimited list of Sponsors: | |
| Crowdfunding goal: | 0 | 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.
|