Use context managers in automation.py

RESOLVED WONTFIX

Status

Testing
General
RESOLVED WONTFIX
6 years ago
6 months ago

People

(Reporter: emorley, Unassigned)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

6 years ago
Broken out from bug 808410.

(In reply to Gregory Szorc [:gps] from comment #21)
> Comment on attachment 681926 [details] [diff] [review]
> Part 5: Callers of checkForCrashes should use its return value
...
> ::: build/automation.py.in
> @@ +1075,5 @@
> >      if os.path.exists(processLog):
> >        os.unlink(processLog)
> >  
> >      if self.IS_TEST_BUILD and runSSLTunnel:
> >        ssltunnelProcess.kill()
> 
> This script looks like it could use some context manager love!
We moved all the useful bits out of automation.py into mozbase.
Status: NEW → RESOLVED
Last Resolved: 6 months ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.