Closed
Bug 1295699
Opened 8 years ago
Closed 7 years ago
TypeError: environment can only contain strings
Categories
(Firefox Build System :: Mach Core, enhancement)
Tracking
(firefox54 fixed)
RESOLVED
FIXED
mozilla54
Tracking | Status | |
---|---|---|
firefox54 | --- | fixed |
People
(Reporter: evilpie, Assigned: blassey)
Details
Attachments
(1 file)
2.39 KB,
patch
|
gps
:
review+
|
Details | Diff | Splinter Review |
When passing --debugger=devenv.exe on Windows, I get this following error: TypeError: environment can only contain strings It's caused by mozrunner/base/runner.py: self.process_handler = subprocess.Popen(cmd, env=self.env). self.env contains one value that is of type unicode instead of str. MOZ_GMP_PATH <type 'str'> c:/Users/tom/mozilla-inbound/obj-x86_64-pc-mingw32\dist\bin\gmp-fake\1.0;c:/Users/tom/mozilla-inbound/obj-x86_64-pc-mingw32\dist\bin\gmp-clearkey\0.1;c:/Users/tom/mozilla-inbound/obj-x86_64-pc-mingw32\dist\bin\gmp-fakeopenh264\1.0 <type 'unicode'>
Assignee | ||
Comment 1•7 years ago
|
||
I hit this, this patch uses the work around that we've landed in some other python files
Assignee: nobody → blassey.bugs
Attachment #8828438 -
Flags: review?(gps)
Updated•7 years ago
|
Attachment #8828438 -
Flags: review?(gps) → review+
Pushed by blassey@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/35ee729f6f1e TypeError: environment can only contain strings, convert environment from unicode before passing to popen r=gps
Comment 3•7 years ago
|
||
sorry had to back this out for a flake8 test failure like https://treeherder.mozilla.org/logviewer.html#?job_id=71162125&repo=mozilla-inbound
Flags: needinfo?(blassey.bugs)
Backout by cbook@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/79363f8a5622 Backed out changeset 35ee729f6f1e for flake8 test failure
Assignee | ||
Updated•7 years ago
|
Flags: needinfo?(blassey.bugs)
Pushed by blassey@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/61dec5fbbb25 TypeError: environment can only contain strings, convert environment from unicode before passing to popen r=gps
Comment 6•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/61dec5fbbb25
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox54:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla54
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
•