- Oct 26, 2020
-
-
Ludovico de Nittis authored
If necessary, in runtime.c we should access the original environment instead of the global one, because it might have been edited. Signed-off-by:
Ludovico de Nittis <ludovico.denittis@collabora.com>
-
- Oct 23, 2020
-
-
Simon McVittie authored
Disable GIO modules without provoking scary warnings (#32) See merge request steam/steam-runtime-tools!156
-
- Oct 22, 2020
-
-
Simon McVittie authored
Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
This lets them access non-public API, and means there's one less thing that can go wrong. The cost is about 500K, which is much smaller than a Steam Runtime. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
The code under test was already moved, but the tests couldn't follow until their executable was statically linked to libsteam-runtime-tools, which it now is. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
We no longer need to, because the test is now linked statically. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
This test exercises internal functions that shouldn't really be visible in the ABI. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Now that GIO_MODULE_DIR has been backported into scout's GLib, we can disable GIO modules completely, instead of loading them but then not using them. This avoids some misleading warnings (#32). This will not be completely effective on non-Debian systems until we also patch scout's GLib to make GIO_MODULE_DIR take precedence over the hard-coded legacy search path /usr/lib/gio/modules. The unit test for this is still in tests/pressure-vessel/utils.c for now. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
- Oct 19, 2020
-
-
Ludovico de Nittis authored
Signed-off-by:
Ludovico de Nittis <ludovico.denittis@collabora.com>
-
Simon McVittie authored
wrap: Set all the env vars before building the locking list See merge request steam/steam-runtime-tools!155
-
Ludovico de Nittis authored
The list of locked environment variables should include all the vars that we decided to want, or avoid. Also use `extra_locked_vars_to_unset` everywhere instead of directly unsetting a variable. Signed-off-by:
Ludovico de Nittis <ludovico.denittis@collabora.com>
-
- Oct 08, 2020
-
-
Ludovico de Nittis authored
Add support for Arch Linux /etc/locale.gen Closes #5 See merge request steam/steam-runtime-tools!151
-
Ludovico de Nittis authored
Fixes #5 Signed-off-by:
Ludovico de Nittis <ludovico.denittis@collabora.com>
-
- Oct 07, 2020
-
-
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
Include STEAMSCRIPT and STEAMSCRIPT_VERSION in the report See merge request steam/steam-runtime-tools!150
-
Ludovico de Nittis authored
These two environment variables can be useful for inferring how and which version of the Steam bootstrapper have been used. For example if the reported version is something like "1.0.0.63-1/Debian", we are quite sure that Steam has been installed using the package from Debian's non-free archive area. Signed-off-by:
Ludovico de Nittis <ludovico.denittis@collabora.com>
-
- Oct 06, 2020
-
-
Simon McVittie authored
Progress towards regenerating ld.so.cache (T14481) See merge request steam/steam-runtime-tools!149
-
Simon McVittie authored
wrap: Do not lock LD_PRELOAD Closes #21 See merge request steam/steam-runtime-tools!146
-
Simon McVittie authored
adverb: Don't "pile up" errors if we cannot exec generate-locales See merge request steam/steam-runtime-tools!148
-
Simon McVittie authored
launch: Add support for communicating with Flatpak services See merge request steam/steam-runtime-tools!147
-
Simon McVittie authored
We can use this to run other helpers, such as ldconfig. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
This means we can modify our own environment without affecting child processes. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
We want to reinstate the old signal mask unconditionally, but we only want to throw an error for it if we weren't already about to throw an error. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
This lets us work around flatpak-spawn not closing file descriptors (https://github.com/flatpak/flatpak-xdg-utils/pull/37 ) by bypassing it, if we want to. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Ludovico de Nittis authored
If we update and install new packages, without doing an "upgrade", there might be some incompatibility issues. Signed-off-by:
Ludovico de Nittis <ludovico.denittis@collabora.com>
-
Ludovico de Nittis authored
For LD_PRELOAD we want to use the value `-launcher` might be receiving from `-launch`, so we don't want to lock it's value in `wrap`. Signed-off-by:
Ludovico de Nittis <ludovico.denittis@collabora.com>
-
Simon McVittie authored
Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
- Oct 05, 2020
-
-
Simon McVittie authored
launch.c: Close the file descriptors after being forwarded See merge request steam/steam-runtime-tools!145
-
Ludovico de Nittis authored
g_unix_fd_list_append() duplicates the given FD, so we need to close our copy, otherwise it might be left indefinitely open. Also, as soon as we send them over D-Bus, we need to unref the FD list to avoid keeping them open. Signed-off-by:
Ludovico de Nittis <ludovico.denittis@collabora.com>
-
- Oct 02, 2020
-
-
Simon McVittie authored
This sometimes takes longer than 30 seconds when several packages are built in parallel on OBS. 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
launcher: Always set PWD to the command's actual working directory See merge request steam/steam-runtime-tools!143
-
- Oct 01, 2020
-
-
Simon McVittie authored
pressure-vessel-wrap unsets PWD, but we don't want the command to inherit a value of PWD from the launcher. In particular, when running in session mode, the launcher's $PWD is the Steam installation, typically ~/.local/share/Steam, and the setup commands are also run with that working directory, but the actual game is run with the current working directory set to its own game directory (which we didn't necessarily even know at the time that the launcher was started). Consumers of $PWD should really check that it is equivalent to the actual current working directory and ignore it if it does not, like GNU get_current_dir_name(3) and our pv_get_current_dirs() do, but blindly believing $PWD is a common shell-scripting mistake, and I wouldn't be surprised if there are some games whose launcher scripts will believe $PWD even when it doesn't coincide with the real working directory. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Add an environment variables locking system Closes #17 See merge request steam/steam-runtime-tools!134
-
Ludovico de Nittis authored
If we want to pass all the possible variables that a user might have set in its environment, we need a mechanism to lock the variables that pressure-vessel is editing (e.g. LIBGL_DRIVERS_PATH), variables that we want to keep unset because will be wrong in the new container (e.g. FLATPAK_ID) and variables that should inherit their value from the host system (e.g. DISPLAY). Signed-off-by:
Ludovico de Nittis <ludovico.denittis@collabora.com>
-
- Sep 30, 2020
-
-
Simon McVittie authored
Signed-off-by:
Simon McVittie <smcv@collabora.com>
-