- Nov 11, 2019
-
-
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>
-
Simon McVittie authored
Fix window system aliasing/normalization (#6) Closes #6 See merge request steam/steam-runtime-tools!80
-
Simon McVittie authored
Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
When a window system/rendering interface pair is an alias for another, we normalize the resulting SrtGraphics object. This test passes when cherry-picked to a commit prior to b3aa7bee, regressed in b3aa7bee, and is fixed by 1b7990b8. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
The intended design for SrtGraphics was for the "X11" window system for the GL family of rendering interfaces to be an alias for either GLX (for "desktop" GL) or EGL_X11 (for GLESv2, and GLESv1 if we implemented it), but this was lost during refactoring. Fixes: b3aa7bee "Pull argv creation out to helper function" Resolves: #6 Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
- Nov 07, 2019
-
-
Ludovico de Nittis authored
Detect graphics library vendor See merge request steam/steam-runtime-tools!79
-
Ludovico de Nittis authored
With this change we are now able to check if the used driver is GLVND or non-GLVND. This information is useful for example when steam-runtime-system-info is failing to find GLX ICDs. When this will happens we will be able to know if this is due to the use of non-GLVND driver or if there is something unexpected. Signed-off-by:
Ludovico de Nittis <ludovico.denittis@collabora.com>
-
- Nov 05, 2019
-
-
Simon McVittie authored
Look for vulkaninfo, wflinfo in SRT_HELPERS_PATH See merge request steam/steam-runtime-tools!77
-
Simon McVittie authored
This allows steam-runtime-tools to be built as a Debian package with nearly full functionality for as-yet-unsupported architectures, such as the ARM family. Builds using the upstream build system won't be fully functional without something like -Dmultiarch_tuple=arm-linux-gnueabi (so maybe don't use a Raspberry Pi as your primary development machine yet), and there is no well-known constant SRT_ABI_xxx for non-x86 architectures. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
This will help us to locate those executables in the Steam Runtime usr/bin directory, even when run without that directory in PATH. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Previously, we always looked for vulkaninfo and wflinfo in the PATH. Now we look in the helpers directory, and only search the PATH if the necessary executable isn't there. This enables us to ship relative symlinks from the helpers directory /usr/libexec/steam-runtime-tools-0 to an adjacent /usr/bin directory, so that if the Steam Runtime is relocated to (for example) .../steam-runtime, we will find vulkaninfo and wflinfo in .../steam-runtime/usr/bin. 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
We have two broad classes of helper: - Helpers from this source package, normally found in /usr/libexec/steam-runtime-tools-0 - Tools we have repurposed from other source packages (wflinfo and vulkaninfo) found via the $PATH For the former class, our behaviour is unchanged. To support the latter class, a caller can now set the SEARCH_PATH flag. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
This is basically the same as !46. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Instead of using a single helper path, return an argv-style array to which the caller can append their own arguments. This will allow us to absorb support for prepending things like timeout(1). Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
- Nov 04, 2019
-
-
Simon McVittie authored
The GLESv2 and GL cases for _argv_for_graphics_test() are essentially the same, so take advantage of that. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
This gets it out of the way of a future function that takes a helpers_path argument. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
This function initially always returned a non-NULL path even in error conditions, but having structured error handling in all callers lets us make it "failable". Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Previously we used CANNOT_LOAD here, but that seems wrong. If we can run check-vulkan, but it immediately crashes, we would use CANNOT_DRAW; let's use the same result if we cannot run it. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
CANNOT_LOAD is a misleading result for an invalid symbols file format. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
- Nov 01, 2019
-
-
Jeremy Whiting authored
Fix check-vulkan when installed to final location See merge request steam/steam-runtime-tools!76
-
Simon McVittie authored
This avoids them having identical detached debug symbols in different binary packages, which cannot be co-installed. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
We install them in /usr/libexec/steam-runtime-tools-0/shaders, not in /usr/libexec/steam-runtime-tools-0. Signed-off-by:
Simon McVittie <smcv@collabora.com>
-
Simon McVittie authored
Signed-off-by:
Simon McVittie <smcv@collabora.com>
-