Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(513)

Unified Diff: tools/utils.py

Issue 2984653002: Really disable core dump archiving on Linux (Closed)
Patch Set: Created 3 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/utils.py
diff --git a/tools/utils.py b/tools/utils.py
index ac06c60d22589cf4c49bb9d3702e89eb22a916f6..88f17a0b89e31981d30d0070ed4335402da99c96 100644
--- a/tools/utils.py
+++ b/tools/utils.py
@@ -726,16 +726,18 @@ class PosixCoreDumpEnabler(object):
def __exit__(self, *_):
resource.setrlimit(resource.RLIMIT_CORE, self._old_limits)
+# TODO(whesse): Re-enable after issue #30205 is addressed
class LinuxCoreDumpEnabler(PosixCoreDumpEnabler):
def __enter__(self):
+ pass
# Bump core limits to unlimited if core_pattern is correctly configured.
- if CheckLinuxCoreDumpPattern(fatal=False):
- super(LinuxCoreDumpEnabler, self).__enter__()
+ # if CheckLinuxCoreDumpPattern(fatal=False):
+ # super(LinuxCoreDumpEnabler, self).__enter__()
def __exit__(self, *args):
- # TODO(whesse): Re-enable after issue #30205 is addressed
+ pass
# CheckLinuxCoreDumpPattern(fatal=True)
- super(LinuxCoreDumpEnabler, self).__exit__(*args)
+ # super(LinuxCoreDumpEnabler, self).__exit__(*args)
class WindowsCoreDumpEnabler(object):
"""Configure Windows Error Reporting to store crash dumps.
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698