This is the mail archive of the cygwin-apps mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [ITP] grepmail 5.3033 -- search mailboxes for mail matching an expression


* Sat 2007-12-01 Dr Dr <Dr.Volker.Zell-QHcLZuEGTsvQT0dZR+AlfA@public.gmane.org>
* Message-Id: 82mysujibo.fsf@vzell-de.de.oracle.com
> But /usr/lib/perl5/site_perl/5.8/cygwin/auto/grepmail/.packlist contains...

Fixed.

> Also /usr/bin/grepmail contains:
>
>    #!/usr/bin/perl5.8.8 -w
>
>    eval 'exec /usr/bin/perl5.8.8 -w -S $0 ${1+"$@"}'
>        if 0; # not running under some shell

Fixed. Here is new one:

  wget        \
    http://cygwin.cante.net/grepmail/setup.hint \
    http://cygwin.cante.net/grepmail/grepmail-5.3033-1-src.tar.bz2 \
    http://cygwin.cante.net/grepmail/grepmail-5.3033-1.tar.bz2

> According to /usr/share/doc/Cygwin/grepmail-5.3033.README:
>
>   Additional Perl CPAN modules needed. Run this command before starting the
>   program:
>
>     $ perl -MCPAN -e 'install $_ for @ARGV' \
>       Date::Parse Mail::Mbox::MessageParser FileHandle::Unget
>
>
> Given that nobody :-) reads README's wouldn't it be better to also
> package this dependencies ?

The code contains module checks and will instruct how the missing CPAN
modules are installed. See

    CYGWIN-PATCHES/0001-grepmail-Add-CPAN-check.patch

I didn't expect the user to check README :-)

I don't think we should package CPAN, because Perl language itself
contains the needed interface, upgrade and other and dependency checks.

Once again, thanks for eagle eye
Jari

-- 
Welcome to FOSS revolution: we fix and modify until it shines


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]