INSTALL: note Plack and URI::Escape are required at the moment

They really shouldn't be...  Also, it seems like eliminating IPC::Run
is not going to be worth the effort.

diff --git a/INSTALL b/INSTALL
index 6e02b4df5dc0bf5a5dd0002bcfe28e9d86cb0de8..3fe0e4f270a13220603a2cece4537c27b873d748 100644 (file)
--- a/INSTALL
+++ b/INSTALL
@@ -37,6 +37,14 @@ Beyond that, there is a long list of Perl modules required, starting with:
 * Encode::MIME::Header          deb: libencode-perl
                                 rpm: perl-Encode
 
 * Encode::MIME::Header          deb: libencode-perl
                                 rpm: perl-Encode
 

+* Plack                         deb: libplack-perl
+                                rpm: perl-Plack, perl-Plack-Test,
+                                (for HTML/Atom generation)
+
+* URI::Escape                   deb: liburi-perl
+                                rpm: perl-URI
+                                (for HTML/Atom generation)
+

 Where "deb" indicates package names for Debian-derived distributions and
 "rpm" is for RPM-based distributions (only known to work on Fedora).
 
 Where "deb" indicates package names for Debian-derived distributions and
 "rpm" is for RPM-based distributions (only known to work on Fedora).
 


@@ -46,14 +54,6 @@ Numerous optional modules are likely to be useful as well:
                                rpm: perl-Socket6
                                (for IPv6 support)
 
                                rpm: perl-Socket6
                                (for IPv6 support)
 

-  - Plack                      deb: libplack-perl
-                               rpm: perl-Plack, perl-Plack-Test,
-                               (for HTML/Atom generation)
-
-  - URI::Escape                deb: liburi-perl
-                               rpm: perl-URI
-                               (for HTML/Atom generation)
-

   - Search::Xapian             deb: libsearch-xapian-perl
                                rpm: perl-Search-Xapian
                                (for NNTP service or gzipped mbox over HTTP)
   - Search::Xapian             deb: libsearch-xapian-perl
                                rpm: perl-Search-Xapian
                                (for NNTP service or gzipped mbox over HTTP)


@@ -102,12 +102,10 @@ Numerous optional modules are likely to be useful as well:
 
 On Fedora systems, you'll probably also end up wanting
 perl-Test-HTTP-Server-Simple, perl-Devel-Peek, and perl-IPC-Run to run the
 
 On Fedora systems, you'll probably also end up wanting
 perl-Test-HTTP-Server-Simple, perl-Devel-Peek, and perl-IPC-Run to run the

-test suite.  On Debian systems, libxml-feed-perl and libipc-run-perl(*)
+test suite.  On Debian systems, libxml-feed-perl and libipc-run-perl

 will aid in running the test suite (XML::Feed and IPC::Run respectively,
 on CPAN).
 
 will aid in running the test suite (XML::Feed and IPC::Run respectively,
 on CPAN).
 

-(*) we hope to drop this dependency someday
-

 standard MakeMaker installation (Perl)
 --------------------------------------
 
 standard MakeMaker installation (Perl)
 --------------------------------------
 

diff --git a/Makefile.PL b/Makefile.PL
index 027c3e63fb742a09a5090ec569d9f35f6c997da6..c134ff92893db27a83d7b6e637f5272f0e369adc 100644 (file)
--- a/Makefile.PL
+++ b/Makefile.PL
@@ -23,10 +23,13 @@ WriteMakefile(
                'Email::MIME::ContentType' => 0,
                'Email::Simple' => 0,
                'Encode::MIME::Header' => 0,
                'Email::MIME::ContentType' => 0,
                'Email::Simple' => 0,
                'Encode::MIME::Header' => 0,

+
+               # TODO: these should really be made optional...

                'Plack' => 0,
                'URI::Escape' => 0,
                'Plack' => 0,
                'URI::Escape' => 0,

+

                # We have more test dependencies, but do not force
                # We have more test dependencies, but do not force

-               # users to install them
+               # users to install them.  See INSTALL

        },
 );
 
        },
 );