Bug 219934 – [Flatpak SDK] coredumpctl invocation when using a local SDK repo fallbacks to the remote repo

• Status: RESOLVED
• Resolution: FIXED
• Component: Tools / Tests

[Lauro Moura]

While investigating bug219933, building the updated SDK followed by running `update-webkit-flatpak`, running a test that crashes shows the following message:

```
Flatpak package org.webkit.Sdk.Debug/x86_64/0.3 not installed. Please update your SDK: Tools/Scripts/update-webkit-flatpak
```

Digging a bit deeper, indeed org.webkit.Sdk.Debug was not being shown in the output of `flatpak list` when generating the list of FlatpakPackages in flatpakutils.py, despite executing the same command in the shell[1] shows the Debug component (org.webkit.Sdk.Locale is present though).

The issue is that linux_get_crash_log.py's _get_trace_from_flatpak invokes webkit-flatpak on the host without keeping the WEBKIT_FLATPAK_USER_DIR variable, making it fall back to the regular remote repo scheme. In my case, installing the remote packages. Due to the silent and lengthy process, after a ctrl+c this remote clone ended incomplete, thus the missing org.webkit.Sdk.Debug message.

WIP patch already working. Will submit after cleanup.

[1] `flatpak list --user --columns=application,arch,branch,origin -a`

[Lauro Moura]

Created attachment 417004 [details]
Patch

[Philippe Normand]

Comment on attachment 417004 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=417004&action=review

> Tools/Scripts/webkitpy/port/linux_get_crash_log.py:117
> +        user_dir = os.environ.get('WEBKIT_FLATPAK_USER_DIR')
> +        if user_dir:
> +            cmd.append("--env=WEBKIT_FLATPAK_USER_DIR=%s" % user_dir)

Might be simpler to pass-through the env to popen?

[Lauro Moura]

(In reply to Philippe Normand from comment #2)
> Comment on attachment 417004 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=417004&action=review
> 
> > Tools/Scripts/webkitpy/port/linux_get_crash_log.py:117
> > +        user_dir = os.environ.get('WEBKIT_FLATPAK_USER_DIR')
> > +        if user_dir:
> > +            cmd.append("--env=WEBKIT_FLATPAK_USER_DIR=%s" % user_dir)
> 
> Might be simpler to pass-through the env to popen?

It's already passed through env as it's popen's default behavior to inherit the environment.

The issue is that flatpak-spawn needs to be instructed which "non default" variables needs to be exported when creating the new shell for the new host command.

[Lauro Moura]

Committed r271199: <https://trac.webkit.org/changeset/271199>

[Radar WebKit Bug Importer]

<rdar://problem/72849449>