Re: Latest CVS on windows

will wrote:
> If this is an issue with file_path as set in the fileentry.py module (line
> 100 or thereabouts), maybe we should just add a line:
>    file_path = file_path.replace(os.sep, "/")

It is not just file_path, it's every other variable that gives the file 
path info, there's a few of those.

> because we want to make sure we're using HTTP separators and not os
> separators.

You also have to think of the variables, if they are reused in a plugin, 
etc.. You may not know what could be affected.

> I just tested it out on my local CVS version (on Linux) and it doesn't
> adversely affect me.  I think this would fix the issue.

Heheh, but of course, the code does nothing to linux users :)

> Does that sound like the right thing to do?

One reason why blosxom can get away with this is due to the fact that 
perl can use '/' as a path seperator and that seperator sticks 
throughout the running system. Python tries to correct the seperator for 
you.. So there. It's not just about file_path, it's category, url, and 
many others I can't just name without looking at the source code.. If I 
don't care about what may happen, I could even do the transform in the 
template parser, but that'll affect every variable out there.

-- 
Regards: Wari Wahab
RoughingIT - http://roughingit.subtlehints.net
PyBlosxom  - http://roughingit.subtlehints.net/pyblosxom



-------------------------------------------------------
This SF.Net email sponsored by: Free pre-built ASP.NET sites including
Data Reports, E-commerce, Portals, and Forums are available now.
Download today and enter to win an XBOX or Visual Studio .NET.
http://aspnet.click-url.com/go/psa00100006ave/direct;at.asp_061203_01/01