conan-center-bot
commented
5 days ago Conan v1 pipeline :x:
Failure in build 1 (9b360b27556d0df4cfcf76c47cc4b97b2b2a7ca8):
-
bgfx/1.127.8771: CI failed to create some packages (All logs)
Logs for packageID
``` [settings] arch=x86_64 build_type=Release compiler=gcc compiler.libcxx=libstdc++11 compiler.version=9 os=Linux [options] bgfx:shared=False ``` ``` [...] ----Running------ > cmake --build '/home/conan/workspace/prod-v1/bsr/cci-ceb0f923/recipes/bgfx/consolidated/test_v1_package/build/ff828b48c3d218c9c72b050ec1a6c750acf541f4' '--' '-j3' ----------------- Scanning dependencies of target test_package [ 50%] Building CXX object test_package/CMakeFiles/test_package.dir/test_package.cpp.o CMake Warning: Manually-specified variables were not used by the project: CMAKE_EXPORT_NO_PACKAGE_REGISTRY CMAKE_INSTALL_BINDIR CMAKE_INSTALL_DATAROOTDIR CMAKE_INSTALL_INCLUDEDIR CMAKE_INSTALL_LIBDIR CMAKE_INSTALL_LIBEXECDIR CMAKE_INSTALL_OLDINCLUDEDIR CMAKE_INSTALL_SBINDIR In file included from /home/conan/workspace/prod-v1/bsr/60890/fecaf/.conan/data/bgfx/1.127.8771/_/_/package/4159b958656e503d491ea2d672d42b805f007389/include/bx/bx.h:15, from /home/conan/workspace/prod-v1/bsr/cci-ceb0f923/recipes/bgfx/consolidated/test_package/test_package.cpp:1: /home/conan/workspace/prod-v1/bsr/60890/fecaf/.conan/data/bgfx/1.127.8771/_/_/package/4159b958656e503d491ea2d672d42b805f007389/include/bx/platform.h:467:27: error: static assertion failed: ** IMPORTANT! ** C++17 standard support is required to build. 467 | static_assert(__cplusplus >= BX_LANGUAGE_CPP17, "\n\n" | ^ make[2]: *** [test_package/CMakeFiles/test_package.dir/build.make:82: test_package/CMakeFiles/test_package.dir/test_package.cpp.o] Error 1 make[1]: *** [CMakeFiles/Makefile2:113: test_package/CMakeFiles/test_package.dir/all] Error 2 make: *** [Makefile:103: all] Error 2 WARN: *** Conan 1 is legacy and on a deprecation path *** WARN: *** Please upgrade to Conan 2 *** bgfx/1.127.8771 (test package): WARN: ************************************************ The 'cmake_find_package_multi' generator is deprecated. Please update your code and remove it. ************************************************* bgfx/1.127.8771 (test package): WARN: ************************************************ The 'cmake' generator is deprecated. Please update your code and remove it. ************************************************* bgfx/1.127.8771 (test package): WARN: **** The 'from conans import CMake' helper is deprecated. Please update your code and remove it. **** ERROR: bgfx/1.127.8771 (test package): Error in build() method, line 12 cmake.build() ConanException: Error 2 while executing cmake --build '/home/conan/workspace/prod-v1/bsr/cci-ceb0f923/recipes/bgfx/consolidated/test_v1_package/build/ff828b48c3d218c9c72b050ec1a6c750acf541f4' '--' '-j3' ```4159b958656e503d491ea2d672d42b805f007389:
Note: To save resources, CI tries to finish as soon as an error is found. For this reason you might find that not all the references have been launched or not all the configurations for a given reference. Also, take into account that we cannot guarantee the order of execution as it depends on CI workload and workers availability.
Conan v2 pipeline :x:
Note: Conan v2 builds are now mandatory. Please read our discussion about it.
The v2 pipeline failed. Please, review the errors and note this is required for pull requests to be merged. In case this recipe is still not ported to Conan 2.x, please, ping @conan-io/barbarians on the PR and we will help you.
Failure in build 1 (9b360b27556d0df4cfcf76c47cc4b97b2b2a7ca8):
-
bgfx/1.127.8771: CI failed to create some packages (All logs)
Logs for packageID
``` [settings] arch=x86_64 build_type=Release compiler=apple-clang compiler.cppstd=17 compiler.libcxx=libc++ compiler.version=13 os=Macos [options] */*:shared=False ``` ``` [...] dxgi.cpp glcontext_egl.cpp glcontext_html5.cpp glcontext_wgl.cpp nvapi.cpp renderer_agc.cpp renderer_d3d11.cpp renderer_d3d12.cpp renderer_gl.cpp renderer_gnm.cpp renderer_mtl.mm renderer_nvn.cpp renderer_noop.cpp ../../../src/bgfx.cpp:2645:87: warning: converting the result of '?:' with integer constants to a boolean always evaluates to 'true' [-Wtautological-constant-compare] { gl::rendererCreate, gl::rendererDestroy, BGFX_RENDERER_OPENGL_NAME, !!BGFX_CONFIG_RENDERER_OPENGLES }, // OpenGLES ^ ../../../src/config.h:99:8: note: expanded from macro 'BGFX_CONFIG_RENDERER_OPENGLES' ? BGFX_CONFIG_RENDERER_OPENGLES_MIN_VERSION : 0) ^ ../../../src/config.h:88:54: note: expanded from macro 'BGFX_CONFIG_RENDERER_OPENGLES_MIN_VERSION' # define BGFX_CONFIG_RENDERER_OPENGLES_MIN_VERSION (0 \ ^ renderer_vk.cpp shader.cpp shader_dxbc.cpp shader_spirv.cpp topology.cpp vertexlayout.cpp ../../../src/renderer_vk.cpp:6520:31: warning: unused variable 'blockInfo' [-Wunused-variable] const bimg::ImageBlockInfo &blockInfo = bimg::getBlockInfo(tf); ^ ../../../src/renderer_mtl.mm:469:3: warning: "XCode 15 is required for visionOS" [-W#warnings] # warning "XCode 15 is required for visionOS" ^ ../../../src/renderer_mtl.mm:2012:17: fatal error: no type or protocol named 'MTLBinding' , NSArrayc88da71a16597ac0b609ab2fd85224adbcd05299:
Note: To save resources, CI tries to finish as soon as an error is found. For this reason you might find that not all the references have been launched or not all the configurations for a given reference. Also, take into account that we cannot guarantee the order of execution as it depends on CI workload and workers availability.
RazielXYZ
Summary
Changes to recipe: bgfx
Motivation
This adds a consolidated recipe for bgfx going forward. This is versioned using bgfx' versioning scheme (https://github.com/bkaradzic/bgfx/blob/93961afcfd45bdcbdb91bddfb133a621d340f136/src/bgfx.cpp#L3566), and integrates bx and bimg into this package. This was done after discussions about whether having bx and bimg separate is valuable, and how due to their interdependence, need for dependency sources, and reliance on specific versions, having them separate makes updates on CCI much slower and more cumbersome.
Details
The recipe is still based on the previous bgfx recipe, but with quite a few changes and additions. It adds all things required for bimg and bx to be integrated into this recipe, such as defines, libs, and their headers. It also adds an option for rtti, due to the fact that bgfx' (and bx', and bimg's) build scripts are hardcoded to build with rtti disabled. Packages on CCI are rtti enabled by default, and mixing these can cause linker issues, especially on linux in my experience. There is also improved support for mingw and android, changes to make other things more consistent, and so on. The new recipe has been locally tested on windows 11 msvc194 and mingw, linux gcc, and android.
Some questions I was pondering: Would there be value in adding an option for overriding bx' CRT detection in order for this to build with something like musl? Would there be value in exposing options for opengl version etc. and other compile-time options that bgfx has?