Closed
Bug 1384665
Opened 7 years ago
Closed 7 years ago
mach check-spidermonkey dies running config/check_spidermonkey_style.py
Categories
(Firefox Build System :: General, defect)
Tracking
(firefox56 fixed)
RESOLVED
FIXED
mozilla56
Tracking | Status | |
---|---|---|
firefox56 | --- | fixed |
People
(Reporter: jimb, Assigned: gps)
References
Details
Attachments
(1 file)
When I run 'mach check-spidermonkey', when it gets to the check_spidermonkey_style.py step, it fails for me as follows: (I commented out the earlier steps, running jit-tests, jstests, and jsapi-tests. I hope those don't have necessary side effects.) shibui:dbg$ mach check-spidermonkey running check-style Traceback (most recent call last): File "/home/jimb/moz/dbg/config/check_spidermonkey_style.py", line 45, in <module> from mozversioncontrol import get_repository_from_env ImportError: No module named mozversioncontrol running check-masm Traceback (most recent call last): File "/home/jimb/moz/dbg/config/check_macroassembler_style.py", line 30, in <module> from mozversioncontrol import get_repository_from_env ImportError: No module named mozversioncontrol running check-js-msg-encoding Traceback (most recent call last): File "/home/jimb/moz/dbg/config/check_js_msg_encoding.py", line 17, in <module> from mozversioncontrol import get_repository_from_env ImportError: No module named mozversioncontrol Error running mach: ['check-spidermonkey'] The error occurred in the implementation of the invoked mach command. This should never occur and is likely a bug in the implementation of that command. Consider filing a bug for this issue. If filing a bug, please include the full output of mach, including this error message. The details of the failure are as follows: NameError: global name 'jittest_result' is not defined File "/home/jimb/moz/dbg/testing/mach_commands.py", line 448, in run_checkspidermonkey all_passed = jittest_result and jstest_result and jsapi_tests_result and check_style_result and check_masm_result and check_js_msg_result
Assignee | ||
Comment 1•7 years ago
|
||
This is fallout from recent work of mine. I'll code up a fix.
Comment hidden (mozreview-request) |
Assignee | ||
Comment 3•7 years ago
|
||
Also, that NameError in your error output has me perplexed. The static analyzer in my brain tells me that can't occur with the current code. Were you hacking up code in testing/mach_commands.py?
Assignee | ||
Comment 4•7 years ago
|
||
Oh, your initial comment mentions commenting out. So, yes.
Comment hidden (mozreview-request) |
Reporter | ||
Comment 6•7 years ago
|
||
mozreview-review |
Comment on attachment 8890634 [details] Bug 1384665 - Use virtualenv Python in `mach check-spidermonkey`; https://reviewboard.mozilla.org/r/161784/#review167122 More carefully choosing the Python interpreter seems like a clear win.
Attachment #8890634 -
Flags: review?(jimb) → review+
Pushed by gszorc@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/54f3aaacf9e6 Use virtualenv Python in `mach check-spidermonkey`; r=jimb
Comment 8•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/54f3aaacf9e6
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox56:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
Updated•6 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•