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

Re: CVS webwml/english/News/weekly



Tollef Fog Heen wrote:
> * Martin Schulze 
> 
> | Debian CVS tfheen wrote:
> |
> | >  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.
> 
> It is used for fixing up the header, not lynx' link output.

Hmm, ok, I understand.  But it doesn't work either...

Calling: ./makemail.pl 2001/17/index.en.html

Without:

Debian Weekly News
2001/17/index.en.html
Debian Weekly News - July 18th, 2001

Within:

---------------------------------------------------------------------------
Debian Weekly News
2001/17/index.html
Debian Weekly News - July 18th, 2001
---------------------------------------------------------------------------

Should be:

---------------------------------------------------------------------------
Debian Weekly News
http://www.debian.org/News/weekly/2001/17/
Debian Weekly News - July 18th, 2001
---------------------------------------------------------------------------

I've fixed that now.

> You Can't Win

How true...

Regards,

	Joey

-- 
Experience is something you don't get until just after you need it.



Reply to: