ping, anyone?

On Fri, Apr 21, 2023 at 9:13 AM Abel Abraham Camarillo Ojeda <
acam...@verlet.org> wrote:

> Updates lensfun to 0.3.3. Hundreds of new cameras and lenses.
>
> make test passes: 100% tests passed, 0 tests failed out of 8
> Total Test time (real) =   8.72 sec
>
> Verified with rawtherapee that it shows my previously unsupported camera
> and lenses
>
> I request review of patch-tests_test_modifier_cpp i know nothing of C++.
> Everything else seems straightforward.
>
> Thanks
>
> patch inline and attached (gmail, possibly mangled)
>
> Index: Makefile
> ===================================================================
> RCS file: /cvs/ports/graphics/lensfun/Makefile,v
> retrieving revision 1.31
> diff -u -p -u -p -r1.31 Makefile
> --- Makefile 11 Mar 2022 19:22:39 -0000 1.31
> +++ Makefile 21 Apr 2023 15:11:08 -0000
> @@ -1,11 +1,12 @@
>  COMMENT= library to adjust for photographic lens characteristics
> -DISTNAME= lensfun-0.3.2
> -REVISION= 4
> +GH_ACCOUNT= lensfun
> +GH_PROJECT= lensfun
> +GH_TAGNAME= v0.3.3
>
> -SHARED_LIBS= lensfun 2.0
> +SHARED_LIBS= lensfun 2.1
>
>  CATEGORIES= graphics
> -HOMEPAGE= http://lensfun.sourceforge.net/
> +#HOMEPAGE= http://lensfun.sourceforge.net/
>
>  # library LGPLv3, lens db CC-BY-SA, build infrastructure/applications
> GPLv3
>  PERMIT_PACKAGE= Yes
> @@ -14,7 +15,7 @@ WANTLIB += c glib-2.0 m png ${COMPILER_L
>
>  COMPILER = base-clang ports-gcc base-gcc
>
> -MASTER_SITES= ${MASTER_SITE_SOURCEFORGE:=lensfun/}
> +#MASTER_SITES= ${MASTER_SITE_SOURCEFORGE:=lensfun/}
>
>  MODULES= devel/cmake \
>   lang/python
> Index: distinfo
> ===================================================================
> RCS file: /cvs/ports/graphics/lensfun/distinfo,v
> retrieving revision 1.4
> diff -u -p -u -p -r1.4 distinfo
> --- distinfo 31 Jan 2017 09:58:35 -0000 1.4
> +++ distinfo 21 Apr 2023 15:11:08 -0000
> @@ -1,2 +1,2 @@
> -SHA256 (lensfun-0.3.2.tar.gz) =
> rovK1GYUykf1vaZbAK9KJXqVZKYXJd+cdMsmDaVE0zE=
> -SIZE (lensfun-0.3.2.tar.gz) = 784825
> +SHA256 (lensfun-0.3.3.tar.gz) =
> V7paA3fySUiXIznhi+lGrxLtoit8cH6w3dJlhjcPZ2U=
> +SIZE (lensfun-0.3.3.tar.gz) = 992432
> Index: patches/patch-cmake_modules_FindGLIB2_cmake
> ===================================================================
> RCS file:
> /cvs/ports/graphics/lensfun/patches/patch-cmake_modules_FindGLIB2_cmake,v
> retrieving revision 1.2
> diff -u -p -u -p -r1.2 patch-cmake_modules_FindGLIB2_cmake
> --- patches/patch-cmake_modules_FindGLIB2_cmake 11 Mar 2022 19:22:39 -0000
> 1.2
> +++ patches/patch-cmake_modules_FindGLIB2_cmake 21 Apr 2023 15:11:08 -0000
> @@ -1,11 +1,12 @@
>  Avoid messing up between paths found by pkg-config and CMake itself.
> ---- cmake/modules/FindGLIB2.cmake.orig Sun Nov 15 18:07:26 2015
> -+++ cmake/modules/FindGLIB2.cmake Sun Jan 29 10:12:03 2017
> +Index: cmake/modules/FindGLIB2.cmake
> +--- cmake/modules/FindGLIB2.cmake.orig
> ++++ cmake/modules/FindGLIB2.cmake
>  @@ -1,6 +1,6 @@
>   IF (NOT MSVC)
>       INCLUDE(FindPkgConfig)
> --    PKG_SEARCH_MODULE( GLIB2 REQUIRED glib-2.0 )
> -+    PKG_SEARCH_MODULE(PC_GLIB2  REQUIRED glib-2.0 )
> +-    PKG_SEARCH_MODULE( GLIB2 glib-2.0 )
> ++    PKG_SEARCH_MODULE(PC_GLIB2 glib-2.0 )
>       IF(WIN32 AND NOT BUILD_STATIC)
>           FIND_FILE(GLIB2_DLL
>                   NAMES glib-2.dll glib-2-vs9.dll libglib-2.0-0.dll
> Index: patches/patch-tests_test_modifier_cpp
> ===================================================================
> RCS file:
> /cvs/ports/graphics/lensfun/patches/patch-tests_test_modifier_cpp,v
> retrieving revision 1.2
> diff -u -p -u -p -r1.2 patch-tests_test_modifier_cpp
> --- patches/patch-tests_test_modifier_cpp 11 Mar 2022 19:22:39 -0000 1.2
> +++ patches/patch-tests_test_modifier_cpp 21 Apr 2023 15:11:08 -0000
> @@ -3,7 +3,7 @@ Fix c++11-narrowing with clang6
>  Index: tests/test_modifier.cpp
>  --- tests/test_modifier.cpp.orig
>  +++ tests/test_modifier.cpp
> -@@ -77,8 +77,8 @@ void test_mod_projection_center(lfFixture* lfFix, gcon
> +@@ -78,8 +78,8 @@ void test_mod_projection_center(lfFixture* lfFix, gcon
>   // check if output becomes NaN when processing geometry conversion
>   void test_mod_projection_borders(lfFixture* lfFix, gconstpointer data)
>   {
> @@ -14,3 +14,28 @@ Index: tests/test_modifier.cpp
>       float res[2] = {0, 0};
>
>       lfLensType geom_types [] = {LF_RECTILINEAR, LF_PANORAMIC,
> LF_EQUIRECTANGULAR, LF_FISHEYE_STEREOGRAPHIC, LF_FISHEYE,
> LF_FISHEYE_EQUISOLID, LF_FISHEYE_ORTHOGRAPHIC, LF_FISHEYE_THOBY,
> LF_UNKNOWN};
> +@@ -102,18 +102,18 @@ void test_mod_projection_borders(lfFixture* lfFix,
> gco
> +                 LF_MODIFY_GEOMETRY, false);
> +
> +             if (lfFix->mod->ApplyGeometryDistortion(0,0,1,1,res)) {
> +-                g_assert_false(std::isnan(res[0]));
> +-                g_assert_false(std::isnan(res[1]));
> ++                g_assert_false(isnan(res[0]));
> ++                g_assert_false(isnan(res[1]));
> +             }
> +
> +             if
> (lfFix->mod->ApplyGeometryDistortion(in[0],in[1],1,1,res)) {
> +-                g_assert_false(std::isnan(res[0]));
> +-                g_assert_false(std::isnan(res[1]));
> ++                g_assert_false(isnan(res[0]));
> ++                g_assert_false(isnan(res[1]));
> +             }
> +
> +             if
> (lfFix->mod->ApplyGeometryDistortion(in2[0],in2[1],1,1,res)) {
> +-                g_assert_false(std::isnan(res[0]));
> +-                g_assert_false(std::isnan(res[1]));
> ++                g_assert_false(isnan(res[0]));
> ++                g_assert_false(isnan(res[1]));
> +             }
> +
> +             delete lfFix->mod;
>
>

Reply via email to