Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

glslang version 13.0.0 #19277

Closed
wants to merge 8 commits into from
Closed

glslang version 13.0.0 #19277

wants to merge 8 commits into from

Conversation

psyinf
Copy link
Contributor

@psyinf psyinf commented Aug 18, 2023

Specify library name and version: glslang/13.0.0

This introduces version 13.0.0 which did some restructuring and exposes additional headers needed also for this PR.
Unfortunately the patch management concerning the PIC got messy so I decided to move this all to a single patch file

Required by #18996


@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@uilianries
Copy link
Member

@psyinf Thank you for your contribution, it seems like some libraries/components are no longer present in this new version. The Conan hook spotted that mismatch. Could you please check the cpp_info vs what is installed in fact? During the package step, all installed artifacts are listed, so it can give a help.

@uilianries uilianries self-assigned this Sep 7, 2023
@psyinf psyinf changed the title glslang version 12.3.1 glslang version 13.0.0 Sep 8, 2023
@psyinf
Copy link
Contributor Author

psyinf commented Sep 8, 2023

@psyinf Thank you for your contribution, it seems like some libraries/components are no longer present in this new version. The Conan hook spotted that mismatch. Could you please check the cpp_info vs what is installed in fact? During the package step, all installed artifacts are listed, so it can give a help.
@uilianries I removed the components for the new version/shared build, but I honestly don't feel to confident here
After having the component problem out of the way the CI run fails due to an older CMake version in the build job than required by the package. Any hints here?

@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@psyinf
Copy link
Contributor Author

psyinf commented Sep 17, 2023

@conan-io/barbarians Anything I can do about the failing builds due to the cmake version on the build-hosts is too low? Since the version is from the upstream package, there is not much I can do here

@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@conan-center-bot
Copy link
Collaborator

Conan v1 pipeline ❌

Failure in build 12 (4328d3c798702827e3a97757ac57393e446f4eb3):

  • glslang/1.3.239.0:
    All packages built successfully! (All logs)

  • glslang/1.3.236.0:
    All packages built successfully! (All logs)

  • glslang/1.3.216.0:
    All packages built successfully! (All logs)

  • glslang/1.3.224.0:
    All packages built successfully! (All logs)

  • glslang/1.3.231.1:
    Didn't run or was cancelled before finishing

  • glslang/1.3.211.0:
    Didn't run or was cancelled before finishing

  • glslang/13.0.0:
    CI failed to create some packages (All logs)

    Logs for packageID d268931894af4ed937a806fe896eac335e4a8526:
    [settings]
    arch=x86_64
    build_type=Release
    compiler=gcc
    compiler.libcxx=libstdc++11
    compiler.version=7
    os=Linux
    [options]
    glslang:shared=True
    
    [...]
    [ 79%] Building CXX object src/SPIRV/CMakeFiles/SPIRV.dir/InReadableOrder.cpp.o
    [ 80%] Building CXX object src/SPIRV/CMakeFiles/SPIRV.dir/Logger.cpp.o
    [ 82%] Building CXX object src/SPIRV/CMakeFiles/SPIRV.dir/SpvBuilder.cpp.o
    [ 83%] Linking CXX shared library libglslang.so
    [ 83%] Built target glslang
    [ 85%] Building CXX object src/SPIRV/CMakeFiles/SPIRV.dir/SpvPostProcess.cpp.o
    [ 86%] Building CXX object src/SPIRV/CMakeFiles/SPIRV.dir/doc.cpp.o
    [ 88%] Building CXX object src/SPIRV/CMakeFiles/SPIRV.dir/SpvTools.cpp.o
    [ 89%] Building CXX object src/SPIRV/CMakeFiles/SPIRV.dir/disassemble.cpp.o
    [ 91%] Building CXX object src/SPIRV/CMakeFiles/SPIRV.dir/CInterface/spirv_c_interface.cpp.o
    [ 92%] Linking CXX shared library libSPIRV.so
    [ 92%] Built target SPIRV
    [ 94%] Generating /home/conan/w/prod-v1/bsr/13035/cbacd/.conan/data/glslang/13.0.0/_/_/build/d268931894af4ed937a806fe896eac335e4a8526/build/Release/include/glslang/glsl_intrinsic_header.h
    [ 95%] Building CXX object src/StandAlone/CMakeFiles/spirv-remap.dir/spirv-remap.cpp.o
    [ 97%] Building CXX object src/StandAlone/CMakeFiles/glslang-standalone.dir/StandAlone.cpp.o
    src/StandAlone/CMakeFiles/spirv-remap.dir/build.make:75: recipe for target 'src/StandAlone/CMakeFiles/spirv-remap.dir/spirv-remap.cpp.o' failed
    CMakeFiles/Makefile2:405: recipe for target 'src/StandAlone/CMakeFiles/spirv-remap.dir/all' failed
    [ 98%] Linking CXX executable glslang
    [ 98%] Built target glslang-standalone
    Makefile:135: recipe for target 'all' failed
    glslang/13.0.0: 
    CMake Deprecation Warning at CMakeLists.txt:1 (cmake_minimum_required):
      Compatibility with CMake < 3.5 will be removed from a future version of
      CMake.
    
      Update the VERSION argument <min> value or use a ...<max> suffix to tell
      CMake that the project does not need compatibility with older versions.
    
    
    CMake Warning:
      Manually-specified variables were not used by the project:
    
        CMAKE_POLICY_DEFAULT_CMP0042
    
    
    /home/conan/w/prod-v1/bsr/13035/cbacd/.conan/data/glslang/13.0.0/_/_/build/d268931894af4ed937a806fe896eac335e4a8526/src/StandAlone/spirv-remap.cpp:40:10: fatal error: filesystem: No such file or directory
     #include <filesystem>
              ^~~~~~~~~~~~
    compilation terminated.
    make[2]: *** [src/StandAlone/CMakeFiles/spirv-remap.dir/spirv-remap.cpp.o] Error 1
    make[1]: *** [src/StandAlone/CMakeFiles/spirv-remap.dir/all] Error 2
    make[1]: *** Waiting for unfinished jobs....
    make: *** [all] Error 2
    glslang/13.0.0: WARN: Using the new toolchains and generators without specifying a build profile (e.g: -pr:b=default) is discouraged and might cause failures and unexpected behavior
    glslang/13.0.0: WARN: Using the new toolchains and generators without specifying a build profile (e.g: -pr:b=default) is discouraged and might cause failures and unexpected behavior
    glslang/13.0.0: ERROR: Package 'd268931894af4ed937a806fe896eac335e4a8526' build failed
    glslang/13.0.0: WARN: Build folder /home/conan/w/prod-v1/bsr/13035/cbacd/.conan/data/glslang/13.0.0/_/_/build/d268931894af4ed937a806fe896eac335e4a8526/build/Release
    ERROR: glslang/13.0.0: Error in build() method, line 170
    	cmake.build()
    	ConanException: Error 2 while executing cmake --build "/home/conan/w/prod-v1/bsr/13035/cbacd/.conan/data/glslang/13.0.0/_/_/build/d268931894af4ed937a806fe896eac335e4a8526/build/Release" '--' '-j3'
    
  • glslang/8.13.3559:
    Didn't run or was cancelled before finishing

  • glslang/11.7.0:
    Didn't run or was cancelled before finishing

  • glslang/11.6.0:
    Didn't run or was cancelled before finishing

  • glslang/11.5.0:
    Didn't run or was cancelled before finishing

  • glslang/1.3.204.0:
    Didn't run or was cancelled before finishing


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.

@uilianries
Copy link
Member

@psyinf That new version requires C++ Filesystem, that's separated distributed:

https://en.cppreference.com/w/cpp/filesystem#Notes

Please, use the follow workaround to fix it:

https://github.com/conan-io/conan-center-index/blob/master/recipes/metall/all/conanfile.py#L85C48-L85C48

@ghost
Copy link

ghost commented Sep 20, 2023

I detected other pull requests that are modifying glslang/all recipe:

This message is automatically generated by https://github.com/ericLemanissier/conan-center-conflicting-prs so don't hesitate to report issues/improvements there.

@psyinf psyinf closed this Sep 20, 2023
@psyinf
Copy link
Contributor Author

psyinf commented Sep 20, 2023

This seems to be superseeded by #19980

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants