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

opencolorio: add version 2.3.0, update dependencies #19549

Closed
wants to merge 5 commits into from

Conversation

toge
Copy link
Contributor

@toge toge commented Sep 1, 2023

Specify library name and version: opencolorio/*


@ghost
Copy link

ghost commented Sep 1, 2023

I detected other pull requests that are modifying opencolorio/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.

@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.

@conan-center-bot
Copy link
Collaborator

Conan v1 pipeline ✔️

All green in build 5 (1a9e00f1e677f9b0252b2d7bfa169152a7cb9950):

  • opencolorio/2.1.0:
    All packages built successfully! (All logs)

  • opencolorio/1.1.1:
    All packages built successfully! (All logs)

  • opencolorio/2.3.0:
    All packages built successfully! (All logs)

  • opencolorio/2.2.1:
    All packages built successfully! (All logs)


Conan v2 pipeline ❌

Regression: Conan v2 builds are mandatory and they are required for the PR to be merged, because this recipe worked with Conan v2 previously.

The v2 pipeline failed. Please, review the errors and note this will be required for pull requests to be merged in the near future.

See details:

Failure in build 5 (1a9e00f1e677f9b0252b2d7bfa169152a7cb9950):

  • opencolorio/2.1.0:
    Didn't run or was cancelled before finishing

  • opencolorio/1.1.1:
    CI failed to create some packages (All logs)

    Logs for packageID 3df00f42d97b1ae02eb17c4fa14ac0aaee5b27bb:
    [settings]
    arch=x86_64
    build_type=Release
    compiler=msvc
    compiler.cppstd=14
    compiler.runtime=dynamic
    compiler.runtime_type=Release
    compiler.version=193
    os=Windows
    [options]
    */*:shared=False
    
    [...]
      C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\Platform.h(111,15): error C2169: 'log2': intrinsic function, cannot be defined (compiling source file C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\FileFormat3DL.cpp) [C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\build\src\core\OpenColorIO_STATIC.vcxproj]
      C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\Platform.h(111,15): error C2169: 'log2': intrinsic function, cannot be defined (compiling source file C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\FileFormatCCC.cpp) [C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\build\src\core\OpenColorIO_STATIC.vcxproj]
      C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\Platform.h(111,15): error C2169: 'log2': intrinsic function, cannot be defined (compiling source file C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\ExponentOps.cpp) [C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\build\src\core\OpenColorIO_STATIC.vcxproj]
      C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\Platform.h(111,15): error C2169: 'log2': intrinsic function, cannot be defined (compiling source file C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\FileFormatCC.cpp) [C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\build\src\core\OpenColorIO_STATIC.vcxproj]
      C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\Platform.h(111,15): error C2169: 'log2': intrinsic function, cannot be defined (compiling source file C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\FileFormatCDL.cpp) [C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\build\src\core\OpenColorIO_STATIC.vcxproj]
      C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\Platform.h(111,15): error C2169: 'log2': intrinsic function, cannot be defined (compiling source file C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\FileFormatCSP.cpp) [C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\build\src\core\OpenColorIO_STATIC.vcxproj]
      C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\Platform.h(111,15): error C2169: 'log2': intrinsic function, cannot be defined (compiling source file C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\FileFormatHDL.cpp) [C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\build\src\core\OpenColorIO_STATIC.vcxproj]
      C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\Platform.h(111,15): error C2169: 'log2': intrinsic function, cannot be defined (compiling source file C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\FileFormatIridasItx.cpp) [C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\build\src\core\OpenColorIO_STATIC.vcxproj]
      C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\Platform.h(111,15): error C2169: 'log2': intrinsic function, cannot be defined (compiling source file C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\FileFormatIridasLook.cpp) [C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\build\src\core\OpenColorIO_STATIC.vcxproj]
      C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\Platform.h(111,15): error C2169: 'log2': intrinsic function, cannot be defined (compiling source file C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\FileFormatIridasCube.cpp) [C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\build\src\core\OpenColorIO_STATIC.vcxproj]
      C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\Platform.h(111,15): error C2169: 'log2': intrinsic function, cannot be defined (compiling source file C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\FileFormatPandora.cpp) [C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\build\src\core\OpenColorIO_STATIC.vcxproj]
      C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\Platform.h(111,15): error C2169: 'log2': intrinsic function, cannot be defined (compiling source file C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\FileFormatSpi1D.cpp) [C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\build\src\core\OpenColorIO_STATIC.vcxproj]
      C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\Platform.h(111,15): error C2169: 'log2': intrinsic function, cannot be defined (compiling source file C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\FileFormatSpi3D.cpp) [C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\build\src\core\OpenColorIO_STATIC.vcxproj]
      C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\Platform.h(111,15): error C2169: 'log2': intrinsic function, cannot be defined (compiling source file C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\FileFormatSpiMtx.cpp) [C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\build\src\core\OpenColorIO_STATIC.vcxproj]
      C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\Platform.h(111,15): error C2169: 'log2': intrinsic function, cannot be defined (compiling source file C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\FileFormatTruelight.cpp) [C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\build\src\core\OpenColorIO_STATIC.vcxproj]
      C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\Platform.h(111,15): error C2169: 'log2': intrinsic function, cannot be defined (compiling source file C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\FileTransform.cpp) [C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\build\src\core\OpenColorIO_STATIC.vcxproj]
      C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\Platform.h(111,15): error C2169: 'log2': intrinsic function, cannot be defined (compiling source file C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\GpuShaderUtils.cpp) [C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\build\src\core\OpenColorIO_STATIC.vcxproj]
      C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\Platform.h(111,15): error C2169: 'log2': intrinsic function, cannot be defined (compiling source file C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\GpuShaderDesc.cpp) [C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\build\src\core\OpenColorIO_STATIC.vcxproj]
      C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\Platform.h(111,15): error C2169: 'log2': intrinsic function, cannot be defined (compiling source file C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\FileFormatVF.cpp) [C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\build\src\core\OpenColorIO_STATIC.vcxproj]
      C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\Platform.h(111,15): error C2169: 'log2': intrinsic function, cannot be defined (compiling source file C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\LogOps.cpp) [C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\build\src\core\OpenColorIO_STATIC.vcxproj]
      C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\Platform.h(111,15): error C2169: 'log2': intrinsic function, cannot be defined (compiling source file C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\Logging.cpp) [C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\build\src\core\OpenColorIO_STATIC.vcxproj]
      C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\Platform.h(111,15): error C2169: 'log2': intrinsic function, cannot be defined (compiling source file C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\Lut1DOp.cpp) [C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\build\src\core\OpenColorIO_STATIC.vcxproj]
      C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\Platform.h(111,15): error C2169: 'log2': intrinsic function, cannot be defined (compiling source file C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\Lut3DOp.cpp) [C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\build\src\core\OpenColorIO_STATIC.vcxproj]
      C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\Platform.h(111,15): error C2169: 'log2': intrinsic function, cannot be defined (compiling source file C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\MathUtils.cpp) [C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\build\src\core\OpenColorIO_STATIC.vcxproj]
      C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\Platform.h(111,15): error C2169: 'log2': intrinsic function, cannot be defined (compiling source file C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\MatrixOps.cpp) [C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\build\src\core\OpenColorIO_STATIC.vcxproj]
      C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\Platform.h(111,15): error C2169: 'log2': intrinsic function, cannot be defined (compiling source file C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\MatrixTransform.cpp) [C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\build\src\core\OpenColorIO_STATIC.vcxproj]
      C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\Platform.h(111,15): error C2169: 'log2': intrinsic function, cannot be defined (compiling source file C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\OCIOYaml.cpp) [C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\build\src\core\OpenColorIO_STATIC.vcxproj]
      C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\Platform.h(111,15): error C2169: 'log2': intrinsic function, cannot be defined (compiling source file C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\PathUtils.cpp) [C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\build\src\core\OpenColorIO_STATIC.vcxproj]
      C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\Platform.h(111,15): error C2169: 'log2': intrinsic function, cannot be defined (compiling source file C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\Platform.cpp) [C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\build\src\core\OpenColorIO_STATIC.vcxproj]
      C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\Platform.h(111,15): error C2169: 'log2': intrinsic function, cannot be defined (compiling source file C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\Processor.cpp) [C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\build\src\core\OpenColorIO_STATIC.vcxproj]
      C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\Platform.h(111,15): error C2169: 'log2': intrinsic function, cannot be defined (compiling source file C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\src\src\core\Transform.cpp) [C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\build\src\core\OpenColorIO_STATIC.vcxproj]
    
        0 Warning(s)
        36 Error(s)
    
    Time Elapsed 00:00:32.31
    
    opencolorio/1.1.1: ERROR: 
    Package '3df00f42d97b1ae02eb17c4fa14ac0aaee5b27bb' build failed
    opencolorio/1.1.1: WARN: Build folder C:\J2\w\prod-v2\bsr@3\1149\efddb\p\b\openc67dd969ef8f98\b\build
    *********************************************************
    Recipe 'opencolorio/1.1.1' cannot build its binary
    It is possible that this recipe is not Conan 2.0 ready
    If the recipe comes from ConanCenter check: https://conan.io/cci-v2.html
    If it is your recipe, check if it is updated to 2.0
    *********************************************************
    
    ERROR: opencolorio/1.1.1: Error in build() method, line 153
    	cm.build()
    	ConanException: Error 1 while executing
    
  • opencolorio/2.3.0:
    Didn't run or was cancelled before finishing

  • opencolorio/2.2.1:
    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.

Copy link
Contributor

@irieger irieger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As I'm new to this but thought I'd use a pull request that is changing something I'm using, here is my two cents. Feel free to ignore this as I'm not fully understand, was just wondering why version are fixed in the CMake files anyway.

Also couldn't fully understand how approvals are handled here from reading the contribution guides and just got access today as I started working on an upgrade for OpenImageIO on the weekend. To me the changes look sensible and build locally also. But From looking at it and as a user of OCIO this looks good to me and I'd approve it.

# yaml-cpp
# https://github.com/jbeder/yaml-cpp
-find_package(yaml-cpp 0.7.0 REQUIRED)
+find_package(yaml-cpp 0.8.0 REQUIRED)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
+find_package(yaml-cpp 0.8.0 REQUIRED)
+find_package(yaml-cpp REQUIRED)

I have never used the version parameter in cmake myself, but does it make sense to fix the version here? Maybe someone has a reason to overwrite the version of yaml-cpp etc. and as long as there is no hard reason I'd personally not restrict it to a fixed version maybe.

@@ -138,7 +138,7 @@ endif()

# minizip-ng
# https://github.com/zlib-ng/minizip-ng
-find_package(minizip-ng 3.0.7 REQUIRED)
+find_package(minizip 3.0.7 REQUIRED)
+find_package(minizip 3.0.9 REQUIRED)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
+find_package(minizip 3.0.9 REQUIRED)
+find_package(minizip REQUIRED)

As above.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@irieger
I'm sorry for missing your comments.

@ghost ghost mentioned this pull request Oct 11, 2023
3 tasks
@toge toge closed this Oct 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants