- Oct 25, 2023
-
-
Simon McVittie authored
Only launcher.py actually needed this. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
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
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
Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
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>
-
- Oct 23, 2023
-
-
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
Now that we correctly read the emulated /etc/os-release in a FEX-Emu environment, we should also report the real OS somewhere. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Previously, s-r-system-info would inspect the live system looking for host OS information, even if it was meant to be parsing a JSON report and not looking at any live data. This needs to be fixed before we can add diagnostic messages to the OS info part of the JSON report, otherwise diagnostic messages from failing to find information about the host OS would become part of the loaded and re-serialized report, causing the round-trip test to fail. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
In the load/save round-trip test, this requires re-saving the file with --verbose, otherwise these fields will be lost. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Unlike SrtOsRelease and its accessors in SrtSystemInfo, this keeps track of everything we might want to add to the JSON report: all os-release(5) fields (not just the important ones), the source path from which we got the information, and any diagnostic messages we logged while collecting it. Because this is a separate object, it'll be more suitable for representing the various contexts that might have an os-release(5). When using FEX-Emu and pressure-vessel, we can have up to four os-release(5) files: the one that is read by native ARM code, the one in the interpreter root pseudo-overlayfs that is read by x86 code, the one that would be read by native ARM code outside the pressure-vessel container, and the one that would be read by x86 code outside the pressure-vessel container. Representing these as four objects with the same API will result in much less duplication. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
We will want this in order to be able to report whether we have found the "right" os-release when running under FEX-Emu. steamrt/tasks#342 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
Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
This is intentionally not using a SrtSysroot, because for some callers we are listing contents relative to a path that is not, technically, a sysroot. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
- Oct 19, 2023
-
-
Simon McVittie authored
This lets us skip a malloc/free cycle in a couple of places. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
It doesn't actually call anything PV-specific. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
- Oct 18, 2023
-
-
Simon McVittie authored
Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
- Oct 13, 2023
-
-
Simon McVittie authored
Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
- Oct 04, 2023
-
-
Simon McVittie authored
This is not guaranteed to be the case now that we are treating libcrypt as potentially a separate library, which is necessary to support use of libxcrypt. Fixes: 81d0099a "Merge libcapsule/v0.20230928.0" Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
- Sep 19, 2023
-
-
Simon McVittie authored
Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
If we get an assertion failure here, we'll want to know which layer is the one that was not as expected. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
If an assertion fails in one of these, we'll want to know which one has the assertion failure. 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>
-
- Sep 04, 2023
-
-
Simon McVittie authored
Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
If we use this in run.sh in preference to running /sbin/ldconfig directly, then we'll only need to add support for distributions with more than one parallel ldconfig binary (like Exherbo) in one place. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
- Aug 31, 2023
-
-
Simon McVittie authored
This allows doing: meson setup _build meson test -C _build without an intervening `meson compile` step. Previously, this would have failed because the tests require various helpers and fixtures, while some of the code under test also requires helper programs, and some of the code under test *is* a CLI program. For a project this small, there's no real need to distinguish precisely which programs are needed by which test: we can just assume that all tests need all programs, like Autotools would. Includes: https://gitlab.gnome.org/GNOME/libglnx/-/merge_requests/51 Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
- Aug 30, 2023
-
-
Simon McVittie authored
If we don't do this, we won't be able to give the tests proper dependencies on the helper programs that they use. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
If we can't chmod the file, we should say what the current owner and permissions look like, to assist debugging. In particular, this is likely to happen on NTFS or when operating on a directory owned by a different user. For example, if the filesystem is owned by root (steam-runtime#544) we might say: Unable to change mode of ... from 0777 (owner: "root", group: "root", non-member) to 0644: ... or if the file is owned by the current user but the filesystem does not allow chmod anyway (steam-runtime#606) we might say: Unable to change mode of ... from 0755 (owner: current user, group: "users") to 0644: ... Helps: https://github.com/ValveSoftware/steam-runtime/issues/434 Helps: https://github.com/ValveSoftware/steam-runtime/issues/460 Helps: https://github.com/ValveSoftware/steam-runtime/issues/477 Helps: https://github.com/ValveSoftware/steam-runtime/issues/491 Helps: https://github.com/ValveSoftware/steam-runtime/issues/544 Helps: https://github.com/ValveSoftware/steam-runtime/issues/606 Signed-off-by:
Simon McVittie <smcv@collabora.com>
-