Bug 209478

Summary: [GTK][WPE] WEB_PROCESS_CMD_PREFIX and others have stopped working
Product: WebKit Reporter: Chris Lord <clord>
Component: Tools / TestsAssignee: Chris Lord <clord>
Status: RESOLVED FIXED    
Severity: Normal CC: aperez, clopez, mcatanzaro, pnormand, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Chris Lord
Reported 2020-03-24 08:14:20 PDT
This was caused by the switch to Flatpak dependencies, which now sandboxes the environment. These variables need to be white-listed.
Attachments
Patch (1.40 KB, patch)
2020-03-24 08:19 PDT, Chris Lord
no flags
Chris Lord
Comment 1 2020-03-24 08:19:24 PDT
Philippe Normand
Comment 2 2020-03-24 08:22:21 PDT
Comment on attachment 394367 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=394367&action=review > Tools/ChangeLog:6 > + Make sure *_PROCESS_CMD_PREFIX variables are whitelisted in the Flatpak sandbox. An alternative could have been to add this to `env_var_suffixes_to_keep` but I'm also OK with this approach. Explicit is better than implicit usually :)
Chris Lord
Comment 3 2020-03-24 08:25:26 PDT
Comment on attachment 394367 [details] Patch I agree with explicit being better (though maybe I'll disagree when this comes back and bites us again ;)) so leaving as is. It would be good if these variables had 'WEBKIT' somewhere in them I think, if we were going to whitelist the suffix.
Philippe Normand
Comment 4 2020-03-24 08:37:46 PDT
Yeah, or a WEBKIT_ prefix :)
EWS
Comment 5 2020-03-24 08:46:52 PDT
Committed r258910: <https://trac.webkit.org/changeset/258910> All reviewed patches have been landed. Closing bug and clearing flags on attachment 394367 [details].
Radar WebKit Bug Importer
Comment 6 2020-03-24 08:47:14 PDT
Note You need to log in before you can comment on or make changes to this bug.