- May 05, 2021
-
-
Now that pressure-vessel can handle unpacked-directory runtimes we can set this as the default behaviour. Signed-off-by:
Ludovico de Nittis <ludovico.denittis@collabora.com>
-
Simon McVittie authored
Reading the build ID from *-buildid.txt no longer works if we don't have that file, which we don't if we are using --no-include-archives. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Previously, 'pinned_version or version' was giving us something like ./soldier_latest-container-runtime. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
This avoids the possibility that they get out of sync. In particular, previously we always generated ./run-in-$suite and ./run as though self.versioned_directories was true; now we take that parameter into account correctly. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
This avoids duplicating the logic necessary to generate it. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Since commit 42fb977a, run() expects to find the tarball in ./.cache. If we're populating the depot with a newly-built runtime, as we do in order to smoke-test new versions of runtimes, then we need to satisfy that assumption by hardlinking it into .cache unconditionally, the same way 42fb977a downloads tarballs into .cache unconditionally. Similarly, since commit bfb0b34d, we might not want to include the tarball (and its build-ID flag file) in the depot. Only put the tarball in the depot if wanted. The pinned_version for a locally-built runtime is None, but we want to use the same build ID for the locally-built runtime that we put in the directory name, so we need to fall back to runtime.version when writing out the ./run and ./run-in-$suite scripts. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
- May 03, 2021
-
-
Ludovico de Nittis authored
We need to store all the downloaded elements paths because if we use a single "downloaded" variable, it might not contain the expected value because it can get replaced by the subsequent elements that we fetched. Signed-off-by:
Ludovico de Nittis <ludovico.denittis@collabora.com>
-
- Apr 30, 2021
-
-
Ludovico de Nittis authored
populate-depot: Write out mtree(5) manifests for unpacked directories See merge request steamlinuxruntime!39
-
- Apr 29, 2021
-
-
Simon McVittie authored
These are what we'll use after T27130 lands in pressure-vessel. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
This can be used to avoid the double disk space consumption of combining the runtime archive with the deployment. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
This allows multiple versions to coexist for local testing, and is also a step towards avoiding the double disk space consumption of deploying the runtime from an archive. The runtime is now unpacked into a versioned directory by default: the Steam CDN now knows how to handle this efficiently, moving unmodified files from one runtime to the next. Using a versioned name ensures that any locally-added files applied by a user will stay where they're put instead of propagating into updated runtimes. The versioned directory name contains `_platform_`, for example `soldier_platform_0.20210428.0`, both for symmetry with the SDK and so that it will not be deleted by pv-wrap --gc-legacy-runtimes. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Anything that isn't Unicode or contains reserved characters is not going to be distributable through Steam's CDN. For the moment we just check for this and don't do anything about it, but pressure-vessel changes that are currently under development will let us provide these files with more Windows-friendly aliases. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
If any such files exist, they will need special treatment, because the Steam CDN assumes Windows filesystem semantics (case-insensitive, case-preserving). Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
This can be used to reconstitute the metadata or permissions of an unpacked directory, even if they are lost in transmission through the Steam CDN (which assumes all files are executable and does not track modification dates). Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Improve CI, always cache downloaded files See merge request steamlinuxruntime!40
-
Simon McVittie authored
This avoids re-downloading the same file unnecessarily. By default, the files are cached in ./.cache. 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
Fix shellcheck warnings See merge request steamlinuxruntime!38
-
- Apr 28, 2021
-
-
Simon McVittie authored
Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
If we write this as LD_PRELOAD='' then it's more obvious what we're doing here. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Never redirect stdout to the log file for setup commands See merge request steamlinuxruntime!37
-
Ludovico de Nittis authored
If we redirect the stdout to a file, Origin starts to complain that it is not able to find the game it was supposed to launch. This seems to be caused by Steam that does something like this: ``` installation_path = subprocess.check_output(".../_v2-entry-point --verb=run -- .../proton run .../iscriptevaluator.exe --print-installation-path") ``` And if stdout is redirected it doesn't work and complains with: ``` src/clientdll/installscript_posix.cpp (357) : Assertion Failed: strCommonRedistForeignInstallPath != "" ``` We already avoided the redirect of stdout when `PRESSURE_VESSEL_BATCH` was set, but apparently this environment variable is not used when we are in the setup phase of a game. Fixes: https://github.com/ValveSoftware/steam-runtime/issues/402 Signed-off-by:
Ludovico de Nittis <ludovico.denittis@collabora.com>
-
- Mar 16, 2021
-
-
Simon McVittie authored
Separate source and generated depot files, improve documentation See merge request steamlinuxruntime!35
-
- Mar 12, 2021
-
-
Simon McVittie authored
Refer to the bug reporting information and known issues, and replace details of how to activate the scout SDK with a simpler approach. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Information that is appropriate for scout is not necessarily appropriate for soldier and vice versa. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
This avoids the depot being a mixture of source files and generated files. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
- Mar 09, 2021
-
-
Simon McVittie authored
Make pressure-vessel-wrap responsible for unpacking runtimes See merge request steamlinuxruntime!32
-
- Feb 23, 2021
-
-
Simon McVittie authored
This simplifies the startup scripts a lot: deploy-runtime and run-in-steamrt no longer need to exist (although I've left behind stub versions to make it easier to switch between versions during testing), and we are not trying to do locking from shell script code. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
- Feb 17, 2021
-
-
Ludovico de Nittis authored
Don't redirect stdout to log under PRESSURE_VESSEL_BATCH Closes #6 See merge request steamlinuxruntime!31
-
Simon McVittie authored
This avoids redirecting the steam-runtime-system-info machine-readable output, or the output of pre-launch setup commands for Proton games. Resolves: steamlinuxruntime#6 Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
- Jan 13, 2021
-
-
Ludovico de Nittis authored
Unpack runtimes into var/ by default See merge request steamlinuxruntime!29
-
Simon McVittie authored
The log is a variable file, so let's move it into the subdirectory. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
This draws a line between the static parts of SteamLinuxRuntime (managed by Steam) and the runtime-modified parts (managed by SteamLinuxRuntime). Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
- Jan 08, 2021
-
-
Simon McVittie authored
_v2-entry-point: use the relaunch container option by default See merge request steamlinuxruntime!28
-
Ludovico de Nittis authored
The relaunch container option allows us to correctly handle the final environment variables from the main program, because we don't know them in advance, when we create the container for the setup steps. The downside is a potentially increased startup time. Until we enable this by default it would be very difficult to estimate the real impact that this option could have on real world systems. Signed-off-by:
Ludovico de Nittis <ludovico.denittis@collabora.com>
-
- Jan 05, 2021
-
-
Ludovico de Nittis authored
Improve logging See merge request steamlinuxruntime!27
-
Simon McVittie authored
It's distracting to see messages about gameoverlayrenderer all over the place. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-