-
Notifications
You must be signed in to change notification settings - Fork 29.7k
Closed
Labels
P1High-priority issues at the top of the work listHigh-priority issues at the top of the work listpackageflutter/packages repository. See also p: labels.flutter/packages repository. See also p: labels.team-infraOwned by Infrastructure teamOwned by Infrastructure team
Description
Some Linux tests in flutter/packages, notably url_launcher, need default handlers set for common schemes (http, https, file) in order for the tests to work. Trying to run them currently on LUCI fails, as discussed in the PR trying to enable them. (Ignore the xvfb part of the discussion there; it's separate, and already resolved.)
For Cirrus we handled this in the Dockerfile. I tried running the same things in a script on LUCI to see what would happen, and got:
xdg-settings set default-web-browser ...to set default browser:
xdg-settings: unknown desktop environmentxdg-mime default ...to set defaultfile:handler:touch: cannot touch '/home/chrome-bot/.local/share/applications/mimeapps.list': No such file or directory /b/s/w/ir/cache/chrome/chrome/xdg-mime: 780: cannot create /home/chrome-bot/.local/share/applications/mimeapps.list.new: Directory nonexistent
So the tools exist, but don't appear to work. While Docker support is one way to possibly resolve this, it would be significant overkill. All we really need to unblock this is either:
- a way for a script to successfully set
http,https, andfiledefault handlers, or - some handlers for those schemes baked into the image (we don't need generic control of other handlers, and I don't foresee needing more than those three). The handlers can be literally anything, we just need
g_app_info_get_default_for_uri_schemeandg_file_query_default_handlerto return something when the plugin tests query for handlers.
Metadata
Metadata
Assignees
Labels
P1High-priority issues at the top of the work listHigh-priority issues at the top of the work listpackageflutter/packages repository. See also p: labels.flutter/packages repository. See also p: labels.team-infraOwned by Infrastructure teamOwned by Infrastructure team