Summary: | Fresh install of Koha cannot find any dependencies | ||
---|---|---|---|
Product: | Koha | Reporter: | Kyle M Hall (khall) <kyle> |
Component: | Architecture, internals, and plumbing | Assignee: | Kyle M Hall (khall) <kyle> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | blocker | ||
Priority: | P5 - low | CC: | f.demians, jonathan.druart, julian.maurice |
Version: | unspecified | ||
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: | 6679 | ||
Bug Blocks: | |||
Attachments: | Bug 15680 - Fresh install of Koha cannot find any dependencies |
Description
Kyle M Hall (khall)
2016-01-27 14:26:11 UTC
Created attachment 47352 [details] [review] Bug 15680 - Fresh install of Koha cannot find any dependencies Fixing the evals in PerlModules.pm to pass perl critic has caused a regression. If appears that using a block eval will not work here, at least not in its current form. Test Plan: 1) Delete your database 2) Run the web installer 3) Note Koha says your dependencies need installed 4) Apply this patch 5) Refresh the page 6) Koha should see your dependencies are installed Kyle, I reproduce the issue, and the patch fixes it. The weird thing is that the script now passes perlcritic (changes in perlcritic?) I'd suggest to revert 0e356b214eb049a446a8d089d579775aa49287d7 instead of create a new patch, to make the git log explicit. (In reply to Jonathan Druart from comment #2) > Kyle, I reproduce the issue, and the patch fixes it. > The weird thing is that the script now passes perlcritic (changes in > perlcritic?) > I'd suggest to revert 0e356b214eb049a446a8d089d579775aa49287d7 instead of > create a new patch, to make the git log explicit. That is most odd! I'll take care of that reversion! Patch pushed to 3.22.x, will be in 3.22.3 This patch has been pushed to 3.20.x, will be in 3.20.9. |