[Distutils] Proposed bdist_dumb fix

Steven Knight knight@baldmt.com
Fri Nov 22 05:47:06 2002


> >		_, t = os.path.splitdrive(t)
> >		if os.path.isabs(t):
> >		    t = t[1:]
> >                archive_root = os.path.join(self.bdist_dir, t)
> >
> >Directly examining the first character loses on some systems.  Old MacOS
> >path names use a leading separator (:) to specify a *relative* path
> >name, for example.
> 
> Hm, so won't this code still be incorrect on the Mac?  Will it have to
> be something like this?
> 
>     if os.path.isabs(t):
>         if platform == 'macos':
>             t = os.sep + t
>         else:
>             t = t[1:]

You're right.  I paid too much attention to the problem with the test
and didn't think through the solution properly.  Good catch; apologies
for the oversight on my part.

	--SK