[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: translation-check and check_trans (was: Re: Debian WWW CVS: peterk)



On Mon, Jan 15, 2001 at 10:22:01PM +0100, Josip Rodin wrote:
> On Mon, Jan 15, 2001 at 08:49:54PM +0100, NAGY Viktor wrote:
> > > % ./check_trans.pl
> > > Can't locate MIME/Lite.pm in @INC (@INC contains: /usr/lib/perl5/5.005/i386-linux /usr/lib/perl5/5.005 /usr/local/lib/site_perl/i386-linux /usr/local/lib/site_perl /usr/lib/perl5 .) at ./check_trans.pl line 149.
> > > BEGIN failed--compilation aborted at ./check_trans.pl line 149.
> > > 
> > > Um. WTH is that necessary?
> > 
> > Yeah, it does not have any sense to enclose the use MIME::Lite; in
> > if ($opt_m) { }  statement, because it's executed at compile time.
> 
> Hmm, that sounds right. $opt_m should be undefined by default, not set to
> '', too. And the verify_send function didn't check for $opt_m, so it kept
> displaying "list is unknown" for me, which is kinda pointless.
> 
> I've commited these changes, sorry if it broke everything for everyone else :)
> 

Ok, I've checked your changes, it's (almost) ok for me. It's good because we
find again the old behaviour of this script, and peter can use the most
uptodate version. He can even modify it.


But it's bad because the new behaviour is broken. As you removed the 
use MIME::Lite; 
the -m option won't work. But it's ok to remove this line as this perl
module isn't debian-packaged yet. I don't really know what to do about this.

1) Put the module in the CVS ?
$ ls -hl /usr/lib/perl5/MIME/Lite.pm 
-rw-r--r--    1 root     root          80k nov 30 12:35
/usr/lib/perl5/MIME/Lite.pm

It's not sooo big.

2) Rely on another mail library in perl ?

Does someone know one ? It must be MIME compliant.

3) others ?


Bye, Mt.



Reply to: