Message6157

Author elentz
Recipients elentz, otmarhumbel
Date 2010-10-08.12:12:11
SpamBayes Score 7.82949e-08
Marked as misclassified No
Message-id <1286539932.94.0.601748371096.issue1639@psf.upfronthosting.co.za>
In-reply-to
Content
Yep, its all valid path. On the Solaris 10 system I'm using and the Fedora Linux 12 system, they have numerous log file references that speak to vfszip:/a/valid/path. So, we would want everything after the ":", minus the /org/python/core/PySystemState.class, just as the patch does.

So, a question for you. Are you concerned about the leading "/" before the "C:"? I didn't make an exhaustive search of PySystemState.py, but I see the result of the method call being used at least in findRoot which uses it in a File class. At least on 1.5.0_22, a leading slash is okay.

Test:
File testFile = new File("/D:/tmp/Foo.txt");
if(testFile.isFile() && testFile.canRead())
  System.out.println("It is a readable file");
BufferedReader input = new BufferedReader(new FileReader(testFile));
System.out.println(input.readLine());

Outputs:
It is a readable file
This is the contents of Foo.txt  (which is what is in Foo.txt, so it works fine)

Thanks for your care and attention to this issue.
History
Date User Action Args
2010-10-08 12:12:12elentzsetmessageid: <1286539932.94.0.601748371096.issue1639@psf.upfronthosting.co.za>
2010-10-08 12:12:12elentzsetrecipients: + elentz, otmarhumbel
2010-10-08 12:12:12elentzlinkissue1639 messages
2010-10-08 12:12:11elentzcreate