- Jul 18, 2024
-
-
Simon McVittie authored
On x86, the 0'th architecture is hard-coded to be x86_64. On other platforms, there is only one architecture, which is whatever we are compiling for. 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>
-
Ludovico de Nittis authored
pv-runtime: Don't try to mask /etc/vulkan/*.d See merge request !727
-
- Jul 17, 2024
-
-
Simon McVittie authored
Since updating our Flatpak-derived code, every time we start a container we log messages like: N: Not replacing "/etc/vulkan/implicit_layer.d" with tmpfs: Path "/etc" is reserved by the container framework We only import selected directories from /etc into the container's /etc, and none of the directories below /etc/vulkan are among them, so there is actually no need to mask these directories. Silence the warning by not attempting to mask them. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
This was implemented in a way that assumed it would only be enabled on specially-flagged developer systems, making it acceptable for the implementation to be non-robust and make assumptions about the host systems on which it would be enabled; but current Steam betas enable it for the steamwebhelper unconditionally, which would cause serious regressions on non-Debian-derived, non-Arch-derived host OSs. Revert the addition of this feature to avoid regressions. Before reinstating it, we will either need to arrange for Steam to enable it conditionally as previously planned, or make the implementation fail gracefully and harmlessly on non-Debian-compatible host systems. This reverts commit d6a519f5. steamrt/tasks#416 Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
- Jul 09, 2024
-
-
Simon McVittie authored
Now that steam-runtime-tools requires GLib 2.58, all backports older than that no longer conflict with s-r-t's backports (and they're skipped anyway as a result of the version checks). 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
pv-runtime: Add a mechanism to take custom CA certificates from the host See merge request !692
-
Simon McVittie authored
Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
logger: Make it easier to attach a logger to a shell script See merge request !722
-
Simon McVittie authored
There are a few places where we want to log the output of a shell script without having to execute srt-logger wrapped around a second shell script as a subprocess. steam.sh and steamwebhelper.sh already use srt-logger, and bin_steam.sh might follow soon. To achieve this, we need to use either a bash coprocess or at least one fifo, because process substitution only creates a one-way pipe. Currently all the scripts where we want to use the logger are bash scripts (as opposed to pure POSIX sh) so we can still take advantage of other bashisms: process substitution and named redirections. Even when using those bash features, the synchronization required between the logger and its parent is non-trivial, so putting it in a shell library makes it easier to achieve. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
If we could rely on `mktemp -d` being fully portable, then we could do this from shell script, but we probably can't rely on that. Similarly, we probably can't rely on `$XDG_RUNTIME_DIR` being set, or `$TMPDIR` being a filesystem that is able to hold fifo objects. This has the same statically-linked dependencies as srt-logger, so make it part of the srt-logger binary to avoid wasting space. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
This will make it easier to locate within a `$STEAM_RUNTIME`. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Unlike the current logger integration in `steam.sh`, this uses `--background` to avoid conflicts with subreapers (steamrt/tasks#460), and `--sh-syntax` to set environment variables (steamrt/tasks#461). The unit test doesn't assert that the output includes the desired environment variables, because we can't assume that the Journal is always available in our test environments, and it's non-trivial to mock up a terminal file descriptor - but when run on a system with a working Journal, it does correctly set `SRT_LOGGER_USE_JOURNAL=1`. steamrt/tasks#461 Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
This will make it easier to set the correct environment variables when attaching it to a shell script like `steam.sh`. steamrt/tasks#461 Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
This will let us convert the environment modifications into other formats, for example a series of shell `export` statements to be `eval`'d. steamrt/tasks#461 Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
This is intended to be a developer-only feature, so it makes some assumptions about the host system: we assume that there is a Debian-compatible TLS certificate store (as seen on e.g. Debian and Arch) with a CAfile at /etc/ssl/certs/ca-certificates.crt and a CApath at /etc/ssl/certs. There is no attempt to make this as robust as we would usually want for an end-user-facing feature, and it will fail hard (exit with an error) rather than trying to continue in the face of adversity. steamrt/tasks#416 Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Add more device test data See merge request !566
-
Simon McVittie authored
logger: Add --sh-syntax, --background See merge request !721
-
Simon McVittie authored
This is an out-of-tree driver providing an alternative interface to X-Box gamepads. Compared with the in-kernel driver, it adds four pseudo-buttons used to indicate which profile is active, and an optional "misc" axis that combines the two triggers into a rudder. Thanks: @kakra Origin: https://github.com/libsdl-org/SDL/pull/10076 Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Thanks: Jeremy Whiting Origin: https://github.com/libsdl-org/SDL/pull/10077 Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Thanks: Robert Beckett Origin: https://github.com/libsdl-org/SDL/pull/10078 Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Thanks: Robert Beckett Origin: https://github.com/libsdl-org/SDL/pull/10078 Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
For some devices, such as the Steam Deck's built-in Steam Controller, the version number reported via evdev and the version number reported via USB are different. We don't currently use this information anyway, but in case we want to use it in the future, let's include it in our test data. Origin: https://github.com/libsdl-org/SDL/pull/10078 Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Thanks: Ravi Kanodia Origin: https://github.com/libsdl-org/SDL/pull/7962 Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
If a device is positively identified as an accelerometer, pointing stick or clickpad, then we don't need to second-guess it. In practice this does not change the result for any device in our test data, so add some artificial records that exercise this. Origin: https://github.com/libsdl-org/SDL/pull/7837 Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
The props array was too small for the highest property bits to be set, although in practice this didn't matter since only the lower-order bits have a meaning. Make it consistent with all the others. Origin: https://github.com/libsdl-org/SDL/pull/7837 Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
They report C/Z buttons but don't physically have them. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Origin: https://github.com/libsdl-org/SDL/pull/7833 Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
We don't currently use these for anything, but we might start using them as input to our heuristics as part of libsdl-org/SDL#7697. Origin: https://github.com/libsdl-org/SDL/pull/7833 Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
All Bluetooth devices are bus type 0x0005. Origin: https://github.com/libsdl-org/SDL/pull/7833 Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
We don't currently use these in our device-classification heuristic, but it could be a useful input in future. Thanks to Sam Lantinga, Ben Fradella, kevenwyld and schlegp for providing some of these. Origin: https://github.com/libsdl-org/SDL/pull/7833 [Also includes subsequent fixes for unreferenced globals] Origin: https://github.com/libsdl-org/SDL/pull/10110 Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Thanks to Ben Fradella. Origin: https://github.com/libsdl-org/SDL/pull/7828 Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
The comments here disagreed with the actual bytes. According to https://elixir.bootlin.com/linux/v6.3.7/source/drivers/hid/hid-wiimote-modules.c, the Balance Board reports BTN_A and ABS_HAT0X, HAT0Y, HAT1X and HAT1Y. This means the comments here were correct, but the .abs bits shown were in the wrong byte. Matching the Wii U Pro Controller against the same kernel source, it appears to be correct: it's the same representation as a PS3 gamepad, except that it lacks the Z and RZ axes for analogue triggers. Origin: https://github.com/libsdl-org/SDL/pull/7816 Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Some of the test data previously seen here was guessed from kernel source code, and not all of it was correct. The following devices have now been verified against `evemu-describe` output with Linux 6.3 (thanks to Jeremy Whiting for collecting this): - basic Wiimote - buttons - 3-axis accelerometer - infra-red sensor for Sensor Bar location (precise aim) - Motion Plus accessory (3-axis gyroscope) - Nunchuck accessory (joystick, 2 buttons, second 3-axis accelerometer) - Classic Controller accessory (a complete traditional gamepad) Origin: https://github.com/libsdl-org/SDL/pull/7816 Signed-off-by:
Simon McVittie <smcv@collabora.com>
-