Summary: | Packaging scripts get upset with non-Debian perl modules | ||
---|---|---|---|
Product: | Koha | Reporter: | Robin Sheat <robin> |
Component: | Packaging | Assignee: | Robin Sheat <robin> |
Status: | CLOSED FIXED | QA Contact: | Paul Poulain <paul.poulain> |
Severity: | minor | ||
Priority: | P5 - low | CC: | chris, jonathan.druart, paul.poulain, robin |
Version: | 3.8 | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Circulation function: | |||
Attachments: |
Bug 8235 - allow a whitelist of modules that aren't in Debian
Bug 8235 - allow a whitelist of modules that aren't in Debian |
Description
Robin Sheat
2012-06-11 15:03:24 UTC
Created attachment 10222 [details] [review] Bug 8235 - allow a whitelist of modules that aren't in Debian This allows modules that aren't in the main debian repos to be handled by the list-deps script when building a control file. Note: this should go happily into stable versions as well as master. Created attachment 10580 [details] [review] Bug 8235 - allow a whitelist of modules that aren't in Debian This allows modules that aren't in the main debian repos to be handled by the list-deps script when building a control file. Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz> For discusion: Isn't it more clean to add a key in the C4::Installer::PerlDependencies::PERL_DEPS hash (like debian) ? Rather than have to modify 2 files (1 pl and 1 pm) ? Possibly, but these things are pretty rare, I'd hope to not have more than a handful floating around at any one time. QA comment : tiny patch, passes QA patch pushed (without testing, but small patch, sounds logical, high level of "trustability" for author and signoffer !) This was released in 3.10.0 |