google-code-export / appengine-devappserver2-experiment

Automatically exported from code.google.com/p/appengine-devappserver2-experiment
0 stars 0 forks source link

dev_appserver should look at the realpath of py files instead of pyc files #15

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. See http://code.google.com/p/nose-gae/issues/detail?id=61

What is the expected output? What do you see instead?
I expect that I can run tests multiple times.  I can't because the ipaddr 
directory resolves differently each time on Mac OS X.

What version of the product are you using? On what operating system?
SDK v1.7.3 on Mac OS X.

Please provide any additional information below.
Please take my patch or the one in issue 61 of nose gae.  Please also put this 
patch into dev_appserver 1.

Original issue reported on code.google.com by jim@twist.com on 13 Dec 2012 at 1:04

Attachments:

GoogleCodeExporter commented 9 years ago

Original comment by bquin...@google.com on 13 Dec 2012 at 11:26

GoogleCodeExporter commented 9 years ago
Fixed in v0.4.

Original comment by sa...@google.com on 20 Dec 2012 at 3:28

GoogleCodeExporter commented 9 years ago
Looks like a typo made it into the patch.  Sorry for not testing sooner.

Original comment by jim@twist.com on 19 Mar 2013 at 10:15

Attachments:

GoogleCodeExporter commented 9 years ago
Hi Jim, we actually fixed that before committing it.

What version of devappserver2 or the Python SDK are you using that this is 
still a problem?

Original comment by bquin...@google.com on 20 Mar 2013 at 6:28

GoogleCodeExporter commented 9 years ago
No, you broke it before committing it.  The code should be replace('pyc', 'py') 
or replace('.pyc', '.py') not replace('.pyc', 'py').  I'm running devappserver2 
from the 1.7.6 appengine sdk.

The patch is generated from the 1.7.6 prerelease devappserver2 against my 
modified 1.7.6 installed devappserver2.

Original comment by jim@twist.com on 20 Mar 2013 at 4:45

GoogleCodeExporter commented 9 years ago
There are multiple repositories in play here :-)

The definitive one is Google's internal repository and it contains:
  py_file = __file__.replace('.pyc', '.py')

That fix with flow out into the next devappserver2 and SDK releases.

Sorry about that.

Original comment by bquin...@google.com on 20 Mar 2013 at 10:58