This isn't super-helpful, but... James Yu wrote: > This is part of my code that invokes urllib.urlretrieve: > > for i in link2Visit: > localName = i.split('/') > i = i.replace(' ', '%20')
You should use urllib.quote or urllib.quote_plus (the latter replaces spaces with "+" instead of "%20") rather than a half-solution that only escapes one character. > tgtPath = ['d:\\', 'work', 'python', 'grab_n_view'] > localPath = '' > for j in tgtPath: > localPath = os.path.join(localPath, j) You could do something like: tgtPath = os.path.join('d:\\', 'work', 'python', 'grab_n_view') (and don't repeat it on every loop). > localPath = os.path.join(localPath, localName[-1]) What about using os.path.basename() instead of the localName business? > info = urllib.urlretrieve(i, localPath) > > link2Visit stores the url to some photos. > After the script finishes running, I got a pile of incomplete JPG files, > each takes only 413 bytes of my disk space. > Did I miss something before using urllib.urlretrieve ? Try opening one of the files in a text editor. It could be a "404 Not Found" error document or something. -- -- http://mail.python.org/mailman/listinfo/python-list