Closed
Bug 1744665
Opened 3 years ago
Closed 3 years ago
"UnboundLocalError: local variable 'rv' referenced before assignment" in python/mach/mach/logging.py
Categories
(Firefox Build System :: General, defect, P1)
Firefox Build System
General
Tracking
(firefox-esr91 unaffected, firefox95 unaffected, firefox96 unaffected, firefox97 fixed)
RESOLVED
FIXED
97 Branch
| Tracking | Status | |
|---|---|---|
| firefox-esr91 | --- | unaffected |
| firefox95 | --- | unaffected |
| firefox96 | --- | unaffected |
| firefox97 | --- | fixed |
People
(Reporter: dao, Assigned: Gijs)
References
(Blocks 1 open bug, Regression)
Details
(Keywords: regression)
Attachments
(1 file)
I got this error when trying to build on Ubuntu 21.10:
UnboundLocalError: local variable 'rv' referenced before assignment
File "/home/dao/mozilla/unified/python/mozbuild/mozbuild/build_commands.py", line 220, in build
return driver.build(
File "/home/dao/mozilla/unified/python/mozbuild/mozbuild/controller/building.py", line 1106, in build
if self._check_clobber(self.mozconfig, os.environ):
File "/home/dao/mozilla/unified/python/mozbuild/mozbuild/controller/building.py", line 1745, in _check_clobber
self.log(logging.WARNING, "clobber", {"msg": line.rstrip()}, "{msg}")
File "/home/dao/mozilla/unified/python/mach/mach/mixin/logging.py", line 53, in log
self._logger.log(level, format_str, extra={"action": action, "params": params})
File "/usr/lib/python3.9/logging/__init__.py", line 1512, in log
self._log(level, msg, args, **kwargs)
File "/usr/lib/python3.9/logging/__init__.py", line 1589, in _log
self.handle(record)
File "/usr/lib/python3.9/logging/__init__.py", line 1599, in handle
self.callHandlers(record)
File "/home/dao/mozilla/unified/third_party/python/sentry_sdk/sentry_sdk/integrations/logging.py", line 77, in sentry_patched_callhandlers
return old_callhandlers(self, record)
File "/usr/lib/python3.9/logging/__init__.py", line 1661, in callHandlers
hdlr.handle(record)
File "/usr/lib/python3.9/logging/__init__.py", line 952, in handle
self.emit(record)
File "/home/dao/mozilla/unified/python/mozbuild/mozbuild/controller/building.py", line 623, in emit
msg = self.format(record)
File "/usr/lib/python3.9/logging/__init__.py", line 927, in format
return fmt.format(record)
File "/home/dao/mozilla/unified/python/mach/mach/logging.py", line 143, in format
return rv + self.sgr0
Flags: needinfo?(ahochheiden)
Comment 1•3 years ago
|
||
Same here on Arch. I did a recent upgrade of python:
[2021-12-04T16:48:35+0100] [ALPM] upgraded python (3.9.7-2 -> 3.9.9-1)
and I think since then I did not launch mach until today.
| Assignee | ||
Comment 2•3 years ago
|
||
Updated•3 years ago
|
Assignee: nobody → gijskruitbosch+bugs
Status: NEW → ASSIGNED
Pushed by gijskruitbosch@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/26c3e7e5467c
un-bust local builds failing with rv being undefined in mach logging code, r=glandium
Pushed by ncsoregi@mozilla.com:
https://hg.mozilla.org/mozilla-central/rev/cd2d02b3fc82
un-bust local builds failing with rv being undefined in mach logging code, r=glandium a=Aryx
Comment 5•3 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
status-firefox97:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 97 Branch
Updated•3 years ago
|
status-firefox95:
--- → unaffected
status-firefox96:
--- → unaffected
status-firefox-esr91:
--- → unaffected
tracking-firefox95:
? → ---
tracking-firefox96:
? → ---
Flags: needinfo?(ahochheiden)
Comment 6•3 years ago
|
||
Thanks for fixing this. I should have caught it during review.
Apologies for the inconvenience.
Updated•3 years ago
|
Has Regression Range: --- → yes
You need to log in
before you can comment on or make changes to this bug.
Description
•