17 Nov
1998
17 Nov
'98
3:18 a.m.
Barry A. Warsaw wrote:
"GS" == Greg Stein <gstein@lyra.org> writes:
GS> Also, there is a general problem in the code with using GS> os.path.join to construct URLs. A URL is NOT a file path. You GS> don't want to use ":" or "\" if Mailman is run on a different GS> platform. The os.path.join should go away and string concat GS> should be used with "/".
Shouldn't we be using urlparse.urljoin() for these?
-Barry
That would only be necessary if the second part is an unknown (possibly-absolute) URL. I wouldn't think it is necessary in this context.
-g
-- Greg Stein, http://www.lyra.org/