Title: [259038] trunk/Tools
- Revision
- 259038
- Author
- ph...@webkit.org
- Date
- 2020-03-26 03:05:11 -0700 (Thu, 26 Mar 2020)
Log Message
[Flatpak SDK] Crash post-mortem debugging is broken
https://bugs.webkit.org/show_bug.cgi?id=209537
Reviewed by Žan Doberšek.
webkit-flatpak --gdb now properly launches gdb to inspect the last
crash reported to coredumpctl. The -m argument can be used to
select another crash dump.
* flatpak/flatpakutils.py:
(WebkitFlatpak.clean_args):
(WebkitFlatpak.run_in_sandbox):
(WebkitFlatpak.run_gdb):
Modified Paths
Diff
Modified: trunk/Tools/ChangeLog (259037 => 259038)
--- trunk/Tools/ChangeLog 2020-03-26 09:01:53 UTC (rev 259037)
+++ trunk/Tools/ChangeLog 2020-03-26 10:05:11 UTC (rev 259038)
@@ -1,3 +1,19 @@
+2020-03-26 Philippe Normand <pnorm...@igalia.com>
+
+ [Flatpak SDK] Crash post-mortem debugging is broken
+ https://bugs.webkit.org/show_bug.cgi?id=209537
+
+ Reviewed by Žan Doberšek.
+
+ webkit-flatpak --gdb now properly launches gdb to inspect the last
+ crash reported to coredumpctl. The -m argument can be used to
+ select another crash dump.
+
+ * flatpak/flatpakutils.py:
+ (WebkitFlatpak.clean_args):
+ (WebkitFlatpak.run_in_sandbox):
+ (WebkitFlatpak.run_gdb):
+
2020-03-25 Fujii Hironori <hironori.fu...@sony.com>
[Win] lld-link: error: /manifestdependency: is not allowed in .drectve
Modified: trunk/Tools/flatpak/flatpakutils.py (259037 => 259038)
--- trunk/Tools/flatpak/flatpakutils.py 2020-03-26 09:01:53 UTC (rev 259037)
+++ trunk/Tools/flatpak/flatpakutils.py 2020-03-26 10:05:11 UTC (rev 259038)
@@ -472,7 +472,7 @@
self.platform = self.platform.upper()
if self.gdb is None and '--gdb' in sys.argv:
- self.gdb = ""
+ self.gdb = True
self.command = "%s %s %s" % (os.path.join(self.sandbox_source_root,
"Tools/Scripts/run-minibrowser"),
@@ -565,7 +565,9 @@
args[0] = command.replace(self.source_root, self.sandbox_source_root)
if args[0].endswith("build-webkit"):
args.append("--prefix=/app")
- building = os.path.basename(args[0]).startswith("build")
+ building = os.path.basename(args[0]).startswith("build")
+ else:
+ building = False
sandbox_build_path = os.path.join(self.sandbox_source_root, "WebKitBuild", self.build_type)
# FIXME: Using the `run` flatpak command would be better, but it doesn't
@@ -833,7 +835,9 @@
executable, = re.findall(".*Executable: (.*)", stderr)
if self.gdb:
- bargs = ["gdb", executable, "/run/host/%s" % coredump.name] + shlex.split(self.gdb)
+ bargs = ["gdb", executable, "/run/host/%s" % coredump.name]
+ if type(self.gdb) != bool:
+ bargs.extend(shlex.split(self.gdb))
elif self.gdb_stack_trace:
bargs = ["gdb", '-ex', "thread apply all backtrace", '--batch', executable, "/run/host/%s" % coredump.name]
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes
