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

Re: CVS webwml/english/News/weekly



Debian CVS tfheen wrote:
> Update of /cvs/webwml/webwml/english/News/weekly
> In directory klecker:/tmp/cvs-serv22038
> 
> Modified Files:
> 	makemail.pl 
> Log Message:
> Fix up the printed header if this is a local file.
> 
> 
> --- /cvs/webwml/webwml/english/News/weekly/makemail.pl	2000/11/01 18:57:38	1.2
> +++ /cvs/webwml/webwml/english/News/weekly/makemail.pl	2001/07/11 05:59:30	1.3
> @@ -17,6 +17,12 @@
>  my $highlink=0;
>  
>  open (IN, "lynx -dump $url |");
> +if ($url =~ m,\d\d\d\d/\d\d?/,) {
> +     # This is a local URL - fix the output
> +     $url =~ s,^\./,http://www.debian.org/News/weekly/,;
> +     $url =~ s/index\.\w\w\.html$/index.html/;
> +}
> +
>  while (<IN>) {
>  	# We exit this loop once we hit the first divider bar,
>  	# which indicates the end of the newsletter proper.

Can this be removed again since it is useless?  lynx won't output
urls with a ./ part, but instead use file://where/the/file/is... 
I've fixed the later part already.

Regards,

	Joey

-- 
Beware of bugs in the above code; I have only proved it correct,
not tried it.  -- Donald E. Knuth



Reply to: