Bug 26848: Fix Readonly dependency in cpanfile
authorJulian Maurice <julian.maurice@biblibre.com>
Wed, 28 Oct 2020 15:29:16 +0000 (16:29 +0100)
committerJonathan Druart <jonathan.druart@bugs.koha-community.org>
Tue, 12 Jan 2021 10:17:35 +0000 (11:17 +0100)
- It is a required module. If it is missing, Koha will not start
- Set the minimum version to 2.00. While not strictly required, it's the
version that states that "Deprecation of Readonly::XS as a requirement
for fast, readonly scalars is complete", it is available in Debian
oldoldstable (jessie, 8), and Readonly::XS was not even needed with Perl
> 5.8, which allows us to:
- Remove dependency on Readonly::XS

Signed-off-by: Mason James <mtj@kohaaloha.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
cpanfile

index 6f43bf4..b7bddb8 100644 (file)
--- a/cpanfile
+++ b/cpanfile
@@ -92,6 +92,7 @@ requires 'PDF::Table', 'v0.9.3';
 requires 'POSIX', '1.09';
 requires 'Plack::Middleware::LogWarn', '0.001002';
 requires 'Plack::Middleware::ReverseProxy', '0.14';
+requires 'Readonly', '2.00';
 requires 'Schedule::At', '1.06';
 requires 'Search::Elasticsearch', '5.01';
 requires 'Sereal::Decoder', '3.0';
@@ -158,8 +159,6 @@ recommends 'Net::Server', '0.97';
 recommends 'Net::Z3950::SimpleServer', '1.15';
 recommends 'PDF::FromHTML', '0.31';
 recommends 'Parallel::ForkManager', '0.75';
-recommends 'Readonly', '0.01';
-recommends 'Readonly::XS', '0.01';
 recommends 'SMS::Send', '0.05';
 recommends 'Selenium::Remote::Driver', '1.27';
 recommends 'Sys::CPU', '0.52';