Script 'mail_helper' called by obssrc Hello community, here is the log from the commit of package OpenImageIO for openSUSE:Factory checked in at 2024-02-15 21:00:35 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/OpenImageIO (Old) and /work/SRC/openSUSE:Factory/.OpenImageIO.new.1815 (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "OpenImageIO" Thu Feb 15 21:00:35 2024 rev:47 rq:1146674 version:2.5.8.0 Changes: -------- --- /work/SRC/openSUSE:Factory/OpenImageIO/OpenImageIO.changes 2024-02-01 18:07:40.684081854 +0100 +++ /work/SRC/openSUSE:Factory/.OpenImageIO.new.1815/OpenImageIO.changes 2024-02-15 21:01:57.187267409 +0100 @@ -1,0 +2,49 @@ +Wed Feb 7 23:24:15 UTC 2024 - Marcus Rueckert <mrueck...@suse.de> + +- we need at least OpenEXR 2.4. force this + +------------------------------------------------------------------- +Wed Feb 7 22:55:57 UTC 2024 - Marcus Rueckert <mrueck...@suse.de> + +- Upgrade the image files to 20240205T200840 + this fixes a small license issue + +------------------------------------------------------------------- +Wed Feb 7 22:37:34 UTC 2024 - Marcus Rueckert <mrueck...@suse.de> + +- Add https://github.com/AcademySoftwareFoundation/OpenImageIO/pull/4143.patch + Potential patch to fix the aarch64 regressions reported here: + https://github.com/AcademySoftwareFoundation/OpenImageIO/issues/4111 + + This can be dropped in the next version update. + +------------------------------------------------------------------- +Thu Feb 1 22:51:30 UTC 2024 - Marcus Rueckert <mrueck...@suse.de> + +- update to 2.5.8.0: + - feat(oiiotool): New --buildinfo command prints build + information, + - including version, compiler, and all library dependencies. + #4124 + - feat(api): New global getattribute queries: "build:platform", + - "build:compiler", "build:dependencies" #4124 + - feat(openexr): Add support for luminance-chroma OpenEXR images. + #4070 (by jreichel-nvidia) + - fix(raw): Avoid buffer overrun for flip direction cases #4100 + - fix(iv): Assume iv display gamma 2.2 if GAMMA env variable is + not set #4118 + - fix: Fixes to reduce problems identified by static analysis + #4113 + - dev(simd.h): Make all-architecture matrix44::inverse() #4076 + - dev(simd.h): Fix AVX-512 round function #4119 (by AngryLoki) + - build(deps): Remove Findfmt.cmake, rely on that package's + exported config. #4069 #4103 (by Dominik Wójt) + - build(deps): Account for header changes in fmt project trunk + #4109 #4114 + - tests: Shuffle some tests between directories #4091 + - tests: Fix docs test, used wrong namespace #4090 + - docs: Fix broken IBA color management documentation #4104 + - style: Update our formatting standard to clang-format 17.0 and + C++17 #4096 + +------------------------------------------------------------------- Old: ---- OpenImageIO-2.5.7.0.tar.gz oiio-images-20221017T102353.tar.xz New: ---- 4143.patch OpenImageIO-2.5.8.0.tar.gz oiio-images-20240205T200840.tar.xz BETA DEBUG BEGIN: New: - Add https://github.com/AcademySoftwareFoundation/OpenImageIO/pull/4143.patch Potential patch to fix the aarch64 regressions reported here: BETA DEBUG END: ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ OpenImageIO.spec ++++++ --- /var/tmp/diff_new_pack.3Q729e/_old 2024-02-15 21:02:01.399415453 +0100 +++ /var/tmp/diff_new_pack.3Q729e/_new 2024-02-15 21:02:01.403415593 +0100 @@ -31,11 +31,11 @@ %bcond_with apidocs %bcond_with ptex -%define images_ts 20221017T102353 +%define images_ts 20240205T200840 %define so_ver 2_5 %define major_minor_ver 2.5 Name: OpenImageIO -Version: 2.5.7.0 +Version: 2.5.8.0 Release: 0 Summary: Library for Reading and Writing Images License: BSD-3-Clause @@ -46,6 +46,7 @@ Source1: oiio-images-%{images_ts}.tar.xz # NOTE: Please don't uncomment a build requirement unless you have submitted the package to factory and it exists #BuildRequires: Field3D-devel +Patch1: https://github.com/AcademySoftwareFoundation/OpenImageIO/pull/4143.patch BuildRequires: cmake >= 3.12 BuildRequires: dcmtk-devel %if %{with apidocs} @@ -74,7 +75,7 @@ BuildRequires: tbb-devel BuildRequires: txt2man BuildRequires: pkgconfig(OpenColorIO) -BuildRequires: pkgconfig(OpenEXR) +BuildRequires: pkgconfig(OpenEXR) >= 2.4 %if %{with ptex} BuildRequires: ptex-devel-static %endif ++++++ 4143.patch ++++++ >From f0f41e08f88560ed5cc97a8062d1f813b7693068 Mon Sep 17 00:00:00 2001 From: Larry Gritz <l...@larrygritz.com> Date: Wed, 7 Feb 2024 12:01:35 -0800 Subject: [PATCH] fix(simd.h): Address NEON issues Primarily, recent changes (PR 4071) to vint4::store for the NEON case appear to have some type mismatches, which apple clang on ARM-based Mac (including our CI) seems ok with, but which is generating type errors on other ARM Linux platforms. I think the types were weird here, so I tightened it up to get the types right for temporary variables in that function. That's the primary fix here. Secondarily, I modfied simd.h and the CMake setup so that build option USE_SIMD=0 will disable NEON in the same way that it disables SSE. (I realized that USE_SIMD=0 was not disabling NEON, so there was no way for a NEON platform to completely disable SIMD if they needed to.) Signed-off-by: Larry Gritz <l...@larrygritz.com> --- src/cmake/compiler.cmake | 2 +- src/include/OpenImageIO/simd.h | 13 ++++++++++--- 2 files changed, 11 insertions(+), 4 deletions(-) diff --git a/src/cmake/compiler.cmake b/src/cmake/compiler.cmake index 657277e0d3..83d6788731 100644 --- a/src/cmake/compiler.cmake +++ b/src/cmake/compiler.cmake @@ -303,7 +303,7 @@ set (SIMD_COMPILE_FLAGS "") if (NOT USE_SIMD STREQUAL "") message (STATUS "Compiling with SIMD level ${USE_SIMD}") if (USE_SIMD STREQUAL "0") - set (SIMD_COMPILE_FLAGS ${SIMD_COMPILE_FLAGS} "-DOIIO_NO_SSE=1") + set (SIMD_COMPILE_FLAGS ${SIMD_COMPILE_FLAGS} "-DOIIO_NO_SIMD=1") else () string (REPLACE "," ";" SIMD_FEATURE_LIST ${USE_SIMD}) foreach (feature ${SIMD_FEATURE_LIST}) diff --git a/src/include/OpenImageIO/simd.h b/src/include/OpenImageIO/simd.h index 72e771a43d..46c8b392b8 100644 --- a/src/include/OpenImageIO/simd.h +++ b/src/include/OpenImageIO/simd.h @@ -77,6 +77,13 @@ // OIIO_SIMD_HAS_SIMD8 : nonzero if vfloat8, vint8, vbool8 are defined // OIIO_SIMD_HAS_SIMD16 : nonzero if vfloat16, vint16, vbool16 are defined +#ifdef OIIO_NO_SIMD /* Request to disable all SIMD */ +# define OIIO_NO_SSE 1 +# define OIIO_NO_AVX 1 +# define OIIO_NO_AVX2 1 +# define OIIO_NO_NEON 1 +#endif + #if defined(_M_ARM64) || defined(__aarch64__) || defined(__aarch64) # ifndef __ARM_NEON__ # define __ARM_NEON__ @@ -4788,9 +4795,9 @@ OIIO_FORCEINLINE void vint4::store (unsigned char *values) const { _mm_store_ss((float*)values, _mm_castsi128_ps(val8)); #elif OIIO_SIMD_NEON vint4 clamped = m_simd & vint4(0xff); - simd_t val16 = vcombine_s16(vqmovn_s32(clamped), vdup_n_s16(0)); - simd_t val8 = vcombine_u8(vqmovun_s16(val16), vdup_n_u8(0)); - vst1q_lane_u32((uint32_t*)values, val8, 0); + int16x8_t val16 = vcombine_s16(vqmovn_s32(clamped), vdup_n_s16(0)); + uint8x16_t val8 = vcombine_u8(vqmovun_s16(val16), vdup_n_u8(0)); + vst1q_lane_u32((uint32_t*)values, vreinterpretq_u32_u8(val8), 0); #else SIMD_DO (values[i] = m_val[i]); #endif ++++++ OpenImageIO-2.5.7.0.tar.gz -> OpenImageIO-2.5.8.0.tar.gz ++++++ /work/SRC/openSUSE:Factory/OpenImageIO/OpenImageIO-2.5.7.0.tar.gz /work/SRC/openSUSE:Factory/.OpenImageIO.new.1815/OpenImageIO-2.5.8.0.tar.gz differ: char 13, line 1 ++++++ oiio-images-20221017T102353.tar.xz -> oiio-images-20240205T200840.tar.xz ++++++ /work/SRC/openSUSE:Factory/OpenImageIO/oiio-images-20221017T102353.tar.xz /work/SRC/openSUSE:Factory/.OpenImageIO.new.1815/oiio-images-20240205T200840.tar.xz differ: char 15, line 1
