- Jan 24, 2023
-
-
Simon McVittie authored
This makes these scripts a better template to drop into other projects. 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 avoids warnings from newer versions of Meson. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
- Jan 12, 2023
-
-
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
Avoid #!/usr/bin/env bash scripts See merge request !530
-
- Jan 10, 2023
-
-
Simon McVittie authored
Because this script runs inside the SteamLinuxRuntime_soldier container, we don't need to worry about whether /bin/bash exists or makes sense: we know that it should be the bash from Debian 10, which is what this script was written to target. Use it by its absolute path instead of doing a PATH search, to add a bit more predictability. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
These scripts don't use any fancy features from bash, so we can use /bin/sh, which is either bash or something faster (like the dash shell used in Debian and Ubuntu). Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
We are no longer supporting any pressure-vessel releases that didn't. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
bash is not completely portable, so we have to stick to POSIX /bin/sh, which only has one array, "$@". Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
bash is not completely portable, so we have to stick to POSIX /bin/sh, which only has one array, "$@". Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
This is a prerequisite for converting this script to /bin/sh, because this sort of argument vector manipulation would be difficult to do while preserving relative order without access to more than one array. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
This is a prerequisite for converting this script to /bin/sh, because this sort of argument vector manipulation would be difficult to do while preserving relative order without access to more than one array. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Using bash for a shell script is apparently not entirely portable, because #!/bin/bash is not available on some unconventional OSs like NixOS, while the frequently-recommended alternative #!/usr/bin/env bash might find an unusable version of bash in the PATH. In particular, if the bash in the PATH was installed via Nix, then it relies on RUNPATH for correct operation, but the scout runtime's LD_LIBRARY_PATH defeats that. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
This makes it easier to set them up in the wrapper shell scripts without making use of bash-specific features, which are not completely portable. For LD_PRELOAD, the separator is either a colon or a space; for LD_AUDIT, the separator is a colon. There is no facility for escaping the separators. This matches the behaviour of the environment variables in glibc ld.so, as implemented in elf/rtld.c. The usual reason to prefer separate command-line arguments over parsing is that a single merged argument either requires an escape mechanism or cannot represent certain data (for instance loadable modules with a colon in their filenames), but there is no need to try to support things that glibc doesn't, and glibc doesn't support loadable modules with the separator in their filenames anyway. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Parse ELF header for version definitions Closes #95 See merge request !529
-
- Jan 05, 2023
-
-
Ludovico de Nittis authored
Parse the new `missing_versions` and `unexpectedly_unversioned` issues that `inspect-library` could report. 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
If in the `.symbols` file there is a verdef `$VERSION@$VERSION`, we can manually parse the ELF header to check if the library has the expected `$VERSION` or not. All missing versions will be reported in output in a list called `missing_versions`. If the library is unexpectedly unversioned, the `unexpectedly_unversioned` entry will be set to `true`. Signed-off-by:
Ludovico de Nittis <ludovico.denittis@collabora.com>
-
- Jan 04, 2023
-
-
Ludovico de Nittis authored
Move the common print operation in a separate function to reduce code duplication. Signed-off-by:
Ludovico de Nittis <ludovico.denittis@collabora.com>
-
- Dec 15, 2022
-
-
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
If we're using a local or pre-downloaded pressure-vessel-bin.tar.gz, there's no point in trying to download the scout LD_LIBRARY_PATH runtime (for automated testing) from a suite named "local" that doesn't exist. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
- Dec 14, 2022
-
-
Simon McVittie authored
This allows populate-depot.py to learn its own version number when used by unpacking a steam-runtime-tools source tarball, rather than from git, and write that into VERSIONS.txt as the "scripts" component. Now that we've combined SteamLinuxRuntime and s-r-t into one repository, the "scripts" component is somewhat redundant for the soldier and sniper runtimes (since it will always match up with pressure-vessel), but it's still interesting for the scout-on-soldier runtime (which does not contain its own copy of pressure-vessel). Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
We also write this into subprojects/container-runtime so that populate-depot.py can find it there. This will enable populate-depot.py to learn its own version number when used by unpacking a steam-runtime-tools source tarball, rather than from git. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
- Dec 08, 2022
-
-
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
Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
- Dec 07, 2022
-
-
Simon McVittie authored
This cleans up some unnecessary complexity now that we are no longer installing this script into /usr. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
This makes the .deb considerably smaller, around 350K rather than 1.3M. If we want the source code, we can build the .dsc like any other dpkg package. The copy of the source code in /usr used to be useful when we constructed a relocatable pressure-vessel installation by installing it into the Steam Runtime 'scout' SDK and then repacking it from there, but that was rather slow, so our release pipeline is now a lot more direct than that. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
This avoids needing to install a wrapper that has been edited to know its own version number, source directory and so on. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
We won't necessarily have pressure-vessel-relocatable installed when acting on a just-built version. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
This removes a dependency on having configured and unpacked source code available. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Having fewer code paths is a good way to make things more robust, and now that we can prepopulate the cache with the just-built .dsc, we should always be able to do this. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-