@@ -, +, @@ OverDrive 1) Run koha_perl_deps.pl -a, verify that LWP::Protocol::https appears in listing. 1) Remove LWP::Protocol::https (liblwp-protocol-https-perl under Debian). 2) Run an OverDrive search on the OPAC, it should fail. 3) Reinstall LWP::Protocol::https. 4) Rerun OverDrive search, it should now succeed. --- C4/Installer/PerlDependencies.pm | 5 +++++ debian/list-deps | 12 +++++++----- 2 files changed, 12 insertions(+), 5 deletions(-) --- a/C4/Installer/PerlDependencies.pm +++ a/C4/Installer/PerlDependencies.pm @@ -732,6 +732,11 @@ our $PERL_DEPS = { 'required' => '1', 'min_ver' => '0.91', }, + 'LWP::Protocol::https' => { + 'usage' => 'OverDrive integration', + 'required' => '0', + 'min_ver' => '5.836', + }, }; 1; --- a/debian/list-deps +++ a/debian/list-deps @@ -25,7 +25,9 @@ use C4::Installer::PerlDependencies; # These are packages that may not be in the apt archive in a way that # apt-file can find, e.g. in the Koha repo rather than the regular # debian one. -my %overrides = (); +my %overrides = ( + 'LWP::Protocol::https' => 'liblwp-protocol-https-perl|libwww-perl (<6.02), libio-socket-ssl-perl', +); # These are packages we're going to ignore my %ignore = ( @@ -55,7 +57,10 @@ foreach my $module ( keys %$deps ) { @lines = ( @lines, $line ); } } - if ( scalar(@lines) == 1 && $lines[0] ne "" ) { + if ( exists $overrides{$module} ) { + print "$overrides{$module}\n"; + } + elsif ( scalar(@lines) == 1 && $lines[0] ne "" ) { my $pkg = $lines[0]; print "$pkg\n"; } @@ -80,9 +85,6 @@ foreach my $module ( keys %$deps ) { # commit.) # RM note: suspicious? me? always! } - elsif ( exists $overrides{$module} ) { - print "$overrides{$module}\n"; - } elsif ( ! $deps->{$module}->{'required'} ) { # Ignore because we don't have it and we don't care. } --