AcademySoftwareFoundation / OpenColorIO

A color management framework for visual effects and animation.
https://opencolorio.org
BSD 3-Clause "New" or "Revised" License
1.79k stars 455 forks source link

Integration - Port some fixes from main to RB-2.2 - Adsk Contrib #1748

Closed cedrik-fuoco-adsk closed 1 year ago

cedrik-fuoco-adsk commented 1 year ago

Integration PR

The first commit is the version release type name to empty. Each subsequent commit was cherry-picked from the main branch.

These commits correspond to the following PRs: Replace texture2D function with texture for GLSL 1.3 CheckSupportSSE2: Fix sse flags unexpected propagation Adsk contrib - Fix issue with minizip build Adsk contrib - Processor cache does not detect changes in cccid Adsk contrib - Configure the OpenColorIO.pc file on Windows Adsk contrib - Hiding minizip-ng symbols on Mac Adsk contrib - Fix issue with is colorspace linear Adsk Contrib - Improve naming of ICC-based displays on Windows Fix Python ARM wheel Adsk Contrib - Fix inverse Lut1D optimization issue Update documentation for 2.2 release Adsk Contrib - OCIO cmake improvements

cedrik-fuoco-adsk commented 1 year ago

I've created a PR previously but the base branch was wrong. I thought it would clean up the checks by creating a second PR, but the checks from the previous PR can be seen in this PR too since they are from the same branch.

Please ignore the skipped checks , continuous-integration/appveyor/pr — AppVeyor was unable to build non-mergeable pull request and all the duplicated checks as it is coming from a previously created PR which had the wrong base branch.

The CI workflows checks can be seen here: https://github.com/AcademySoftwareFoundation/OpenColorIO/actions/runs/3848353813

sonarcloud[bot] commented 1 year ago

SonarCloud Quality Gate failed.    Quality Gate failed

Bug E 1 Bug
Vulnerability A 0 Vulnerabilities
Security Hotspot E 1 Security Hotspot
Code Smell A 254 Code Smells

56.6% 56.6% Coverage
3.6% 3.6% Duplication

cedrik-fuoco-adsk commented 1 year ago

Closing in favour of PR #1750.