Closed
Bug 503121
Opened 15 years ago
Closed 15 years ago
our version of python for windows mobile handles the command line different than the desktop version
Categories
(Firefox for Android Graveyard :: General, defect)
Firefox for Android Graveyard
General
ARM
Windows Mobile 6 Professional
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: jmaher, Unassigned)
Details
This issue is related to spaces in the commandline string and the fact that we are expecting two arguments (app, args) vs a single commandline argument. I found this while doing xpcshell tests on windows mobile where we set constants on the command line like this: xpcshell -e 'const _HEAD_FILES = ["<path>/head.js", "<path>/head2.js"];' in order to get this to work, I need to do this: xpcshell -e 'var[_HEAD_FILES]=[["<path>/head.js","<path>/head2.js"]];' I suspect the problem is in subprocess.py for the popen handler when it handles 1 commandline vs app/args. We might be able to do more logical parsing of the command line to split these, or rework the createprocess to just one a single command line.
Comment 1•15 years ago
|
||
pushed http://hg.mozilla.org/users/blassey_mozilla.com/pythonce/rev/3e13329ed067 this fixes the problem for all my test cases, if you have any other trouble reopen this bug. binary: http://people.mozilla.org/~blassey/python25.zip debug: http://people.mozilla.org/~blassey/python25-d.zip
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 2•15 years ago
|
||
I have the latest python25.zip and it is missing the pipelib.dll and pipedev.dll. Are these integrated in to another .dll?
Comment 3•15 years ago
|
||
It's staticly linked into python25.dll now
Reporter | ||
Updated•15 years ago
|
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•