Closed Bug 1628748 Opened 4 years ago Closed 4 years ago

Switch to python3 seems to have broken the Compiler database backend

Categories

(Firefox Build System :: General, defect)

defect
Not set
normal

Tracking

(firefox-esr68 unaffected, firefox74 unaffected, firefox75 unaffected, firefox76 unaffected, firefox77 fixed)

RESOLVED FIXED
mozilla77
Tracking Status
firefox-esr68 --- unaffected
firefox74 --- unaffected
firefox75 --- unaffected
firefox76 --- unaffected
firefox77 --- fixed

People

(Reporter: andi, Assigned: andi)

References

(Regression)

Details

(Keywords: regression)

Attachments

(1 file)

Running on macOS:

 ./mach build-backend -b CompileD

Results in:

 0:00.64 /Users/abpostelnicu/Projects/mozilla/obj-ff-osx/_virtualenvs/init_py3/bin/python /Users/abpostelnicu/Projects/mozilla/obj-ff-osx/config.status --backend CompileDB
Reticulating splines...
Traceback (most recent call last):
  File "/Users/abpostelnicu/Projects/mozilla/obj-ff-osx/config.status", line 905, in <module>
    config_status(**args)
  File "/Users/abpostelnicu/Projects/mozilla/mozilla-unified/python/mozbuild/mozbuild/config_status.py", line 148, in config_status
    the_backend.consume(definitions)
  File "/Users/abpostelnicu/Projects/mozilla/mozilla-unified/python/mozbuild/mozbuild/backend/base.py", line 129, in consume
    if (not self.consume_object(obj) and
  File "/Users/abpostelnicu/Projects/mozilla/mozilla-unified/python/mozbuild/mozbuild/compilation/database.py", line 67, in consume_object
    obj.canonical_suffix)
  File "/Users/abpostelnicu/Projects/mozilla/mozilla-unified/python/mozbuild/mozbuild/compilation/database.py", line 213, in _build_db_line
    append_var('OS_COMPILE_CMMFLAGS')
  File "/Users/abpostelnicu/Projects/mozilla/mozilla-unified/python/mozbuild/mozbuild/compilation/database.py", line 204, in append_var
    if isinstance(value, types.string_types):
AttributeError: module 'types' has no attribute 'string_types'

This is because string_types no longer exists in types.

Assignee: nobody → bpostelnicu
Status: NEW → ASSIGNED
Pushed by bpostelnicu@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/8ccb606f2be5
Switch to python3 seems to have broken the Compiler database backend. r=sylvestre
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla77
Has Regression Range: --- → yes
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: