Skip to content

Commit

Permalink
Bug 11757: remove dependency on POE
Browse files Browse the repository at this point in the history
The last use of the POE family of Perl modules went away with
the removal of zebraqueue_daemon.pl per bug 9001.  Consequently,
this patch removes POE as a dependency.

To test:

[1] Verify that "git grep POE" and "git grep libpoe" report
    nothing.
[2] Verify that koha_perl_deps.pl -a does not report POE
    as a dependency.
[3] (extra credit) verify that Debian packages can be built
    that do not list libpoe-perl as a dependency.

This patch also updates some distro-specific installation
instructions and scripts, but makes no representations about
whether those instructions currently work.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
  • Loading branch information
gmcharlt committed Feb 15, 2014
1 parent ef65eee commit b67dac8
Show file tree
Hide file tree
Showing 8 changed files with 0 additions and 13 deletions.
5 changes: 0 additions & 5 deletions C4/Installer/PerlDependencies.pm
Expand Up @@ -89,11 +89,6 @@ our $PERL_DEPS = {
'required' => '1',
'min_ver' => '1.59'
},
'POE' => {
'usage' => 'Core',
'required' => '1',
'min_ver' => '0.9999'
},
'Email::Date' => {
'usage' => 'Core',
'required' => '1',
Expand Down
1 change: 0 additions & 1 deletion INSTALL.fedora7
Expand Up @@ -774,7 +774,6 @@ XML::LibXSLT
XML::Simple
YAML::Syck
Unix::Syslog
POE
Date::Calc
CHI
CHI::Driver::Memcached
Expand Down
1 change: 0 additions & 1 deletion INSTALL.opensuse
Expand Up @@ -857,7 +857,6 @@ The commands show below need to be given one by one, because zypper will always
# zypper install perl-PDF-API2
# zypper install perl-PDF-Reuse
# zypper install perl-PDF-Reuse-Barcode
# zypper install perl-POE
# zypper install perl-32bit
# zypper install perl-SMS-Send
# zypper install perl-Schedule-At
Expand Down
2 changes: 0 additions & 2 deletions debian/control
Expand Up @@ -83,7 +83,6 @@ Build-Depends: libalgorithm-checkdigits-perl,
libpdf-reuse-barcode-perl,
libpdf-reuse-perl,
libpdf-table-perl,
libpoe-perl,
libreadonly-perl,
libreadonly-xs-perl,
libscalar-list-utils-perl,
Expand Down Expand Up @@ -272,7 +271,6 @@ Depends: libalgorithm-checkdigits-perl,
libpdf-reuse-barcode-perl,
libpdf-reuse-perl,
libpdf-table-perl,
libpoe-perl,
libreadonly-perl,
libreadonly-xs-perl,
libscalar-list-utils-perl,
Expand Down
1 change: 0 additions & 1 deletion install_misc/debian.packages
Expand Up @@ -94,7 +94,6 @@ libpdf-api2-simple-perl install
libpdf-reuse-barcode-perl install
libpdf-reuse-perl install
libpdf-table-perl install
libpoe-perl install
libreadonly-perl install
libreadonly-xs-perl install
libschedule-at-perl install
Expand Down
1 change: 0 additions & 1 deletion install_misc/ubuntu.10.04.packages
Expand Up @@ -121,7 +121,6 @@ libpdf-api2-simple-perl install
libpdf-reuse-barcode-perl install
libpdf-reuse-perl install
libpdf-table-perl install
libpoe-perl install
libreadonly-perl install
libreadonly-xs-perl install
libschedule-at-perl install
Expand Down
1 change: 0 additions & 1 deletion install_misc/ubuntu.12.04.packages
Expand Up @@ -126,7 +126,6 @@ libpdf-api2-simple-perl install
libpdf-reuse-barcode-perl install
libpdf-reuse-perl install
libpdf-table-perl install
libpoe-perl install
libreadonly-perl install
libreadonly-xs-perl install
libschedule-at-perl install
Expand Down
1 change: 0 additions & 1 deletion install_misc/ubuntu.packages
Expand Up @@ -126,7 +126,6 @@ libpdf-api2-simple-perl install
libpdf-reuse-barcode-perl install
libpdf-reuse-perl install
libpdf-table-perl install
libpoe-perl install
libreadonly-perl install
libreadonly-xs-perl install
libschedule-at-perl install
Expand Down

0 comments on commit b67dac8

Please sign in to comment.