- Dec 05, 2019
-
-
Simon McVittie authored
This lets us have versioned dependencies on older versions. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
As well as being a harmless and vaguely desirable change in its own right, this works around a bug in deb-build-snapshot, which doesn't always use the dist tarball that it just generated for the updated dpkg package. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Since steam-runtime-tools was made relocatable in c3a06cce, its build will fail with the Steam Runtime 1 'scout' version of dpkg if it is already installed system-wide. If the SDK container where steam-runtime-tools is built has our $(pkglibexecdir) populated with symbolic links to the dependency libraries, then dpkg-shlibdeps will look there to convert DT_NEEDED dependencies into Debian dependencies. Unfortunately, the version of dpkg in SteamRT 1 'scout' has Debian bug #843073, which means it assumes all libraries are found via their canonical names as listed in the dpkg database, not a symlink-based alias. This results in the build failing with an error similar to: dpkg-shlibdeps: error: no dependency information found for /usr/lib/x86_64-linux-gnu/steam-runtime-tools-0/libglib-2.0.so.0 (used by debian/libsteam-runtime-tools-0-helpers/usr/libexec/steam-runtime-tools-0/x86_64-linux-gnu-check-locale) Until we get a newer Docker image with a newer version of this package that works around this issue, we will have to remove it explicitly before running CI. Adding Build-Conflicts is not enough: autopkgtest installs build-dependencies, but does not remove Build-Conflicts. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
- Nov 20, 2019
-
-
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
Add check-gl draw a triangle using opengl test See merge request steam/steam-runtime-tools!83
-
Jeremy Whiting authored
Also added mock-mixed gl test to test scenario of wflinfo working but check-gl failing.
-
Jeremy Whiting authored
Just as with check-vulkan use --visible to see the triangle window otherwise application draws a few frames and reports any gl errors found or does exit 0 on success.
-
- Nov 19, 2019
-
-
Simon McVittie authored
Build -system-info and -check-locale with relative DT_RPATH See merge request steam/steam-runtime-tools!85
-
- Nov 18, 2019
-
-
Simon McVittie authored
Add missing string for cannot draw graphics issue. See merge request steam/steam-runtime-tools!86
-
Jeremy Whiting authored
-
- Nov 15, 2019
-
-
Simon McVittie authored
This ensures that they can find the non-ubiquitous json-glib library, and the steam-runtime-tools library, even if relocated into a non-standard prefix like the LD_LIBRARY_PATH Steam Runtime. We can't just use a relative version of the standard library directory as our RPATH, because if we did that, we would load glibc from the same place. This causes crashes in pressure-vessel, where ld.so and the rest of glibc are often taken from the host system via /overrides/lib, and the glibc in /lib and /usr/lib is incompatible with ld.so. Instead, use a private library directory and populate it with relative symlinks to our non-glibc dependencies; we assume the host version of glibc is new enough that this is OK. Using DT_RPATH in preference to DT_RUNPATH means we take all our direct and indirect dependencies from the same place. This is important when libjson-glib (which won't ordinarily have a RUNPATH or RPATH) pulls in GIO as an indirect dependency of check-locale, which doesn't directly depend on GIO. The GIO library we get must match the versions of GLib and GObject we're using. This also bypasses LD_LIBRARY_PATH. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
- Nov 13, 2019
-
-
Simon McVittie authored
This made them usable from outside the LD_LIBRARY_PATH runtime, but breaks their use inside a pressure-vessel container. In pressure-vessel, the glibc family of libraries are typically taken from outside the container (for example /overrides/lib/x86_64-linux-gnu/libc.so.6), and the dynamic linker ld.so is replaced with one that is suitable for the libraries in /overrides. In general, these versions will be incompatible with the ones in /usr/lib/x86_64-linux-gnu, which unfortunately are the ones that get pulled in via the DT_RPATH. This reverts commit 91478a07. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
- Nov 12, 2019
-
-
Simon McVittie authored
Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Enable and squash compiler warnings See merge request steam/steam-runtime-tools!82
-
- Nov 11, 2019
-
-
Simon McVittie authored
This matters when invoked as a subproject, for example by pressure-vessel, in which case we don't get -I$(top_builddir). Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
This is Meson best-practice: it avoids accidentally including the wrong config.h when using subprojects. 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
As noted in pressure-vessel, if the compiler decides not to inline a `static inline` function (type-safe macro-equivalent), that isn't really a bug. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
We're using GLib, and GLib explicitly doesn't support pedantic Standard C compilers: it assumes and requires the ability to cast between function pointers and data pointers. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
They sometimes give different warnings. 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
In particular this disables -Wmissing-field-initializers in check-vulkan. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
For completeness, I've added separate lists of flags that are only supported for C++, although there are none yet. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
It's inconsistent to ask for this in warning_cflags, and immediately disable it in no_warning_cflags. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Detected by g++ -Wshadow. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Meson warns that get_supported_arguments() is wrong for linker options like these. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Build -system-info and -check-locale with relative DT_RPATH See merge request steam/steam-runtime-tools!78
-
Simon McVittie authored
If steam-runtime-system-info is run from an unpacked LD_LIBRARY_PATH Steam Runtime, we want it to find that runtime's library expectations. In particular, when using "steam-native" on Arch Linux, we want to diagnose whether the system-wide libraries provided by Arch Linux are compatible with the ones from the real Steam Runtime. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
If steam-runtime-system-info is run from an unpacked LD_LIBRARY_PATH Steam Runtime, we want it to find that runtime's library expectations. This is a step towards that. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
- Nov 08, 2019
-
-
Simon McVittie authored
This ensures that they can find the non-ubiquitous json-glib library, and the steam-runtime-tools library, even if relocated into a non-standard prefix like the LD_LIBRARY_PATH Steam Runtime. Using DT_RPATH in preference to DT_RUNPATH means we take all our direct and indirect dependencies from the same place. This is important when libjson-glib (which won't ordinarily have a RUNPATH or RPATH) pulls in GIO as an indirect dependency of check-locale, which doesn't directly depend on GIO. The GIO library we get must match the versions of GLib and GObject we're using. This also bypasses LD_LIBRARY_PATH. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Fix some compiler warnings See merge request steam/steam-runtime-tools!81
-
Simon McVittie authored
VkClearValue is a struct whose first member is a union whose first member is an array of four floats, so we need three levels of braces to initialize it. clang++ 8 warns for this. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
SrtWindowSystem has no valid negative values, so the compiler can use an unsigned base type for it, which makes window_system >= 0 tautologous. SrtRenderingInterface has the same issue. Use an unsigned cast to make the assertion obviously equally valid, whether the enum's base type is signed or not. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-