Closed Bug 1295699 Opened 8 years ago Closed 7 years ago

TypeError: environment can only contain strings

Categories

(Firefox Build System :: Mach Core, enhancement)

x86
Windows 10
enhancement
Not set
normal

Tracking

(firefox54 fixed)

RESOLVED FIXED
mozilla54
Tracking Status
firefox54 --- fixed

People

(Reporter: evilpie, Assigned: blassey)

Details

Attachments

(1 file)

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'>
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)
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
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
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
https://hg.mozilla.org/mozilla-central/rev/61dec5fbbb25
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla54
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: