- Feb 22, 2021
-
-
Simon McVittie authored
Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Ludovico de Nittis authored
Fix handling of Vulkan manifests in /etc (and also error handling) See merge request !245
-
Simon McVittie authored
Add an "issues" entry for ICDs and Vulkan layers See merge request !241
-
Ludovico de Nittis authored
If we don't have duplicated ICDs/Layers in the host system, we want to be sure that this holds true also inside a container. Signed-off-by:
Ludovico de Nittis <ludovico.denittis@collabora.com>
-
Ludovico de Nittis authored
Signed-off-by:
Ludovico de Nittis <ludovico.denittis@collabora.com>
-
Ludovico de Nittis authored
With the additional "issues" entry in the s-r-s-i report we can now flag the ICDs/layers as being duplicated or also unsupported. This makes it easier to spot misconfigured systems or also regressions about how we import ICDs/layers inside the container. Fixes: T26156 Signed-off-by:
Ludovico de Nittis <ludovico.denittis@collabora.com>
-
- Feb 19, 2021
-
-
Simon McVittie authored
In the code path where we copy a file that would not otherwise be visible in the container, the copy doesn't actually work, because glnx_file_copy_at() opens regular files with O_NOFOLLOW and so refuses to read through the "magic symlink", failing with ELOOP. This can be reproduced by having a Vulkan ICD manifest in /etc/vulkan/icd.d/*.json whose `library_path` is just a SONAME. pressure-vessel tries to copy the file, and fails. It turns out glnx_file_copy_at() is not the right function here anyway, because it tries to fchown() the destination file to match the source file, which is going to work poorly if we are copying a file owned by root (which we often are). Just use the important part of it, which is glnx_regfile_copy_bytes(). Resolves: https://github.com/ValveSoftware/steam-runtime/issues/366 Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
_srt_resolve_in_sysroot raises a GError, so it's a bug to set the GError again. Just put an explanatory prefix on the error we already had. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Putting a prefix on the rather opaque error should make it more obvious what we were trying to do. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
If we pass in SRT_RESOLVE_FLAGS_READABLE or SRT_RESOLVE_FLAGS_DIRECTORY, and something goes wrong while we are opening the file, then the best diagnostics libglnx can give us will be something like "openat(/proc/self/fd/21): <strerror result>", which is not very informative. Adding current_path turns this into something more like "Unable to open "foo/bar": openat(/proc/self/fd/21): <strerror result>", which gives us more hope of figuring it out. I originally suspected this was the error path being hit in <https://github.com/ValveSoftware/steam-runtime/issues/366 >, but now I think it probably isn't. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Ludovico de Nittis authored
Signed-off-by:
Ludovico de Nittis <ludovico.denittis@collabora.com>
-
Simon McVittie authored
Add ability to store multiarch tuples See merge request !94
-
Simon McVittie authored
flatpak-run: Update to the latest upstream version Closes #53 See merge request !244
-
Ludovico de Nittis authored
This includes a fix to propagate the X11 cookies that have an address equal to XAUTHLOCALHOSTNAME. Useful especially for OpenSUSE systems. Fixes: #53 Signed-off-by:
Ludovico de Nittis <ludovico.denittis@collabora.com>
-
Ludovico de Nittis authored
Instead of just having a single primary multiarch, now we are able to list multiple multiarch tuples. The first will be considered the primary and all the others will be assumed to be "foreign" architectures. Signed-off-by:
Ludovico de Nittis <ludovico.denittis@collabora.com>
-
- Feb 18, 2021
-
-
Simon McVittie authored
Our Docker runners are currently too old for Arch's glibc. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
build: Use configuration_data instead of manual compiler flags See merge request !243
-
Ludovico de Nittis authored
Meson's `configuration_data()` allows us to generate a configuration file instead of manually passing each options as compiler flags. With this commit we align steam-runtime-tools with pressure-vessel that already used a `config.h.in` file. Signed-off-by:
Ludovico de Nittis <ludovico.denittis@collabora.com>
-
Ludovico de Nittis authored
Start to make use of Flatpak 1.10.1 features See merge request !242
-
- Feb 16, 2021
-
-
Simon McVittie authored
We'll need this if we want to put a Flatpak subsandbox in our process ID namespace, so that Steam and the game can use process-ID-based IPC. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
This was added in Flatpak 1.10.0. 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>
-
- Feb 09, 2021
-
-
Simon McVittie authored
adverb: Remove unused locales temp dir Closes #56 See merge request !240
-
Ludovico de Nittis authored
If we are not going to use the locales temp dir, we remove it immediately. Otherwise it will be left in /tmp/ forever (usually until a system reboot). Fixes: #56 Signed-off-by:
Ludovico de Nittis <ludovico.denittis@collabora.com>
-
- Feb 05, 2021
-
-
Simon McVittie authored
xdg-portal: Replace g_print with the more correct g_debug See merge request !238
-
Ludovico de Nittis authored
Signed-off-by:
Ludovico de Nittis <ludovico.denittis@collabora.com>
-
- Feb 04, 2021
-
-
Ludovico de Nittis authored
pressure-vessel: Accept runtime with or without ./files See merge request !237
-
Simon McVittie authored
Previously, if we were using a Flatpak-style runtime, the argument to --runtime= had to be its ./files subdirectory. Now we accept the top-level directory as an alternative. This will make it more straightforward for pressure-vessel-wrap to take responsibility for unpacking and deploying runtimes that are shipped as a tarball, which is currently done in the SteamLinuxRuntime shell scripts, resulting in error handling and locking being less robust. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Ludovico de Nittis authored
Mask Vulkan layers in home directory or other non-/usr paths See merge request !236
-
Simon McVittie authored
This avoids the Steam fossilize and overlay layers being duplicated in the container's search path. Resolves: T26160 Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
This will break the ability to find .desktop files, etc., in their default search paths. Default to /usr/local/share:/usr/share as per the specification. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
- Feb 03, 2021
-
-
Simon McVittie authored
Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Ludovico de Nittis authored
Revert "runtime: improve ICDs binding speed" See merge request !234
-
Simon McVittie authored
When a JSON manifest in the graphics provider contains an architecture-specific absolute library path (like Mesa in most distributions), we relied on the kind being set to ICD_KIND_NONEXISTENT when we tried the other word size; otherwise, we would create a duplicate JSON manifest, which apparently confuses at least Proton. This reverts commit ee73d5bd. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
- Feb 02, 2021
-
-
Simon McVittie authored
Signed-off-by:
Simon McVittie <smcv@collabora.com>
-