- Oct 25, 2023
-
-
Simon McVittie authored
It doesn't actually have anything to do with pressure-vessel any more. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
There's nothing pressure-vessel-specific about it. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Just for completeness, really. Previously we didn't test this. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Without this, running the test in a soldier container, launched from a host with (for example) LC_ADDRESS=en_GB.utf8, will fail. Until now, this was masked by the fact that this test was unnecessarily skipped when dbus-run-session is unavailable, but I want to change that. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
- Oct 24, 2023
-
-
Simon McVittie authored
Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
steam-runtime-tools v0.20231013.1 * launcher-service: Unblock and unignore all signals during startup, fixing detection of subprocesses exiting when launched by Steam (steamrt/tasks#354, ValveSoftware/SteamVR-for-Linux#618)
-
Simon McVittie authored
-
Simon McVittie authored
Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Steam could run this tool in an arbitrary environment, with necessary signals either blocked (sigprocmask() or similar), or ignored (sigaction() with action SIG_IGN, or similar). Both of these are inherited across fork() and execve() by default, and current versions of Steam do block SIGCHLD in practice. However, blocking or ignoring SIGCHLD breaks handling of subprocesses, which is particularly critical for s-r-launcher-service: that prevents it from detecting that its child processes have exited, causing s-r-launch-client to hang forever, which in turn breaks startup of SteamVR on systems where /bin/sh is dash (but not bash, which apparently resets the disposition of SIGCHLD internally). steamrt/tasks#354, https://github.com/ValveSoftware/SteamVR-for-Linux/issues/618 Signed-off-by:
Simon McVittie <smcv@collabora.com> (cherry picked from commit d2bd4e4b)
-
Simon McVittie authored
urlopen: Use steam://openurl_external to open http, https URLs See merge request !618
-
Simon McVittie authored
Steam could run this tool in an arbitrary environment, with necessary signals either blocked (sigprocmask() or similar), or ignored (sigaction() with action SIG_IGN, or similar). Both of these are inherited across fork() and execve() by default, and current versions of Steam do block SIGCHLD in practice. However, blocking or ignoring SIGCHLD breaks handling of subprocesses, which is particularly critical for s-r-launcher-service: that prevents it from detecting that its child processes have exited, causing s-r-launch-client to hang forever, which in turn breaks startup of SteamVR on systems where /bin/sh is dash (but not bash, which apparently resets the disposition of SIGCHLD internally). steamrt/tasks#354, https://github.com/ValveSoftware/SteamVR-for-Linux/issues/618 Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Ludovico de Nittis authored
Use SrtSysroot in pv-runtime See merge request !616
-
Simon McVittie authored
Convert its test coverage into test coverage for _srt_sysroot_test(), rather than wasting it. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
If the libdir isn't a directory, then we certainly don't need to (and can't) delete its contents. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
No functional change, the mutable sysroot is always accessed via fd-based I/O anyway; but the resulting code is a bit shorter. Several direct uses of _srt_resolve_in_sysroot() remain, because they use SRT_RESOLVE_FLAGS_MKDIR_P, but that's fine. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
As with earlier commits, this will usually have no practical effect, but it will make explicit --graphics-provider=/ more likely to work under FEX-Emu. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
No functional change, because self->host_root is always fd-based I/O anyway. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
This means that if the provider is the current execution environment, we will use path-based rather than fd-based I/O. That should not have any practical effect, because when we're running under FEX-Emu, the graphics provider defaults to the FEX rootfs anyway, and when we are not being emulated, there's no practical difference between path-based and fd-based I/O; but it could matter for non-FEX emulators, and will make explicit --graphics-provider=/ work better under FEX. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
No functional change: self->host_root is never using direct I/O anyway. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
This is necessary for a full replacement for _srt_file_test_in_sysroot(). Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Just for consistency: we know that self->provider->in_current_ns is fdio-based. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
This is only for consistency, and no functional change is intended: we know that the interpreter root is constructed as a fdio-based object. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
We need to be a bit careful here, because we can only interpret dynamic string tokens like `${LIB}` if we're using the real root, which we previously represented as path `/` but now represent with `_srt_sysroot_new_direct()`. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Ludovico de Nittis authored
graphics-drivers: Resolve files in a sysroot more correctly See merge request !602
-
steam://openurl_externalSimon McVittie authored
According to Steam developers, this is better than steam://openurl in some corner cases. It didn't work when I tried it a year ago, but now works in both the beta and GA clients. steamrt/tasks#80 Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Ludovico de Nittis authored
pv-runtime: Use a SrtSysroot to represent the real root See merge request !617
-
Ludovico de Nittis authored
graphics: Write out exactly the original content of Vulkan/EGL driver manifests See merge request !614
-
Ludovico de Nittis authored
system-info: Add path/version of s-r-s-i as the first things in the report See merge request !594
-
Simon McVittie authored
This will let us tell at a glance which version of the diagnostic tool a given report came from. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
If it isn't a directory, then it can't possibly contain files. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
We will want this later, to be able to pass it to interfaces that have been converted to take a SrtSysroot. No functional change. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Otherwise we will miss FEX-Emu's path rewriting. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
These can't possibly contain VDPAU modules. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
This ensures that FEX-Emu's path rewriting is still effective. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-