Message8955

Author pdina
Recipients fwierzbicki, otmarhumbel, pdina, yanne, zyasoft
Date 2014-09-06.23:37:21
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1410046642.31.0.328870350066.issue1672@psf.upfronthosting.co.za>
In-reply-to
Content
In attach a patch that should fix the issue.

1. it's not clear how (where) to correctly override the stdlib module compileall.py. I moved it from /path/to/jython/lib-python/2.7 to /path/to/jython/Lib and changed the file there. Is it correct?

2. This is the change
- cfile = fullname + (__debug__ and 'c' or 'o')
+ cfile = fullname.replace('.py', '$py.class')

When -O flag will be implemented the patch should also consider __debug__.

3. about tests, I should create a temporary directory structure, prepopulated, and delete it at the end of the test. Should I proceed this way?

Thanks
History
Date User Action Args
2014-09-06 23:37:22pdinasetmessageid: <1410046642.31.0.328870350066.issue1672@psf.upfronthosting.co.za>
2014-09-06 23:37:22pdinasetrecipients: + pdina, otmarhumbel, fwierzbicki, zyasoft, yanne
2014-09-06 23:37:22pdinalinkissue1672 messages
2014-09-06 23:37:22pdinacreate