On 03.09.2010 15:08, Stefan Sperling wrote: > On Fri, Sep 03, 2010 at 08:58:09PM +0800, Wei-Yin Chen wrote: >> Sorry, that line should have been >> matching_filenames = [f for f in matching_filenames if not >> os.path.islink(dirname+'/'+f)] > Hi, > > thanks for your patch! > > I think we should use os.sep instead of '/', because os.sep is more portable.
os.path.join maybe? -- Brane
