Script 'mail_helper' called by obssrc Hello community, here is the log from the commit of package libvisual for openSUSE:Factory checked in at 2024-07-14 08:48:41 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/libvisual (Old) and /work/SRC/openSUSE:Factory/.libvisual.new.17339 (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "libvisual" Sun Jul 14 08:48:41 2024 rev:28 rq:1186994 version:0.4.0 Changes: -------- --- /work/SRC/openSUSE:Factory/libvisual/libvisual.changes 2024-03-03 20:19:59.599289124 +0100 +++ /work/SRC/openSUSE:Factory/.libvisual.new.17339/libvisual.changes 2024-07-14 08:48:43.108530824 +0200 @@ -1,0 +2,9 @@ +Fri Jun 14 00:10:03 UTC 2024 - Yasuhiko Kamata <belphe...@belbel.or.jp> + +- Imported C99 compatibility fixes from Fedora project: + https://src.fedoraproject.org/rpms/libvisual/c/bcffd8eddbbcab5b00f930805396be5fdb55c5a7?branch=rawhide + + it fixes build error when using GCC14 (boo#1225859) + (added libvisual-configure-c99.patch and libvisual-c99.patch). + +------------------------------------------------------------------- New: ---- libvisual-c99.patch libvisual-configure-c99.patch BETA DEBUG BEGIN: New: it fixes build error when using GCC14 (boo#1225859) (added libvisual-configure-c99.patch and libvisual-c99.patch). New: it fixes build error when using GCC14 (boo#1225859) (added libvisual-configure-c99.patch and libvisual-c99.patch). BETA DEBUG END: ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ libvisual.spec ++++++ --- /var/tmp/diff_new_pack.5FPxxG/_old 2024-07-14 08:48:44.192570474 +0200 +++ /var/tmp/diff_new_pack.5FPxxG/_new 2024-07-14 08:48:44.192570474 +0200 @@ -1,7 +1,7 @@ # # spec file for package libvisual # -# Copyright (c) 2023 SUSE LLC +# Copyright (c) 2024 SUSE LLC # # All modifications and additions to the file contributed by third parties # remain the property of their copyright owners, unless otherwise agreed @@ -35,6 +35,8 @@ Patch3: %name-%{version}-unref-static.diff Patch4: libvisual-0.4.0-2.1-nmu.diff Patch5: libvisual-0.4.0-inlinedefineconflict.patch +Patch6: libvisual-configure-c99.patch +Patch7: libvisual-c99.patch BuildRequires: freeglut-devel BuildRequires: gcc-c++ BuildRequires: libdrm-devel @@ -77,6 +79,10 @@ %patch -P 3 %patch -P 4 -p1 %patch -P 5 -p1 +%patch -P 6 -p1 +%patch -P 7 -p1 +# Prevent re-building the autotools scripts. +touch -r aclocal.m4 configure* %build autoreconf -fiv ++++++ libvisual-c99.patch ++++++ Includ "lv_cpu.h" for the visual_cpu_initialize, visual_cpu_get_sse, visual_cpu_get_3dnow functions. Add a prototype for visual_transform_init, so that it can be called before it is defined. Upstream has switched to C++, which does not support implicit function declarations at all, so the issue has already been fixed there. diff --git a/libvisual/lv_libvisual.c b/libvisual/lv_libvisual.c index 8de03ebad7d499bb..e7ef768d421aa7fa 100644 --- a/libvisual/lv_libvisual.c +++ b/libvisual/lv_libvisual.c @@ -30,6 +30,7 @@ #include <gettext.h> #include "lvconfig.h" +#include "lv_cpu.h" #include "lv_plugin.h" #include "lv_actor.h" #include "lv_input.h" diff --git a/libvisual/lv_math.c b/libvisual/lv_math.c index cfae316d0abc8c0d..f55c11854051f145 100644 --- a/libvisual/lv_math.c +++ b/libvisual/lv_math.c @@ -29,6 +29,7 @@ #include <math.h> #include "lv_bits.h" +#include "lv_cpu.h" #include "lv_math.h" /* This file is getting big and bloated because of the large chunks of simd code. When all is in place we'll take a serious diff --git a/libvisual/lv_transform.c b/libvisual/lv_transform.c index c763768c0fbb757a..58fe15fa57d2e783 100644 --- a/libvisual/lv_transform.c +++ b/libvisual/lv_transform.c @@ -35,6 +35,8 @@ #include "lv_transform.h" #include "lv_mem.h" +int visual_transform_init (VisTransform *transform, const char *transformname); + extern VisList *__lv_plugins_transform; static int transform_dtor (VisObject *object); ++++++ libvisual-configure-c99.patch ++++++ Avoid implicit function declartions in the configure script, to improve compatibility with future compilers. The âcheck_me != 42â part is not generic and would have to be upstreamed, but upstream has switched to CMake and no longer uses this test, it seems. diff --git a/aclocal.m4 b/aclocal.m4 index 14cd8f3560139cbb..ab8170ee0185c72a 100644 --- a/aclocal.m4 +++ b/aclocal.m4 @@ -1655,10 +1655,6 @@ else # endif #endif -#ifdef __cplusplus -extern "C" void exit (int); -#endif - void fnord() { int i=42;} int main () { @@ -1672,7 +1668,7 @@ int main () /* dlclose (self); */ } - exit (status); + return status; }] EOF if AC_TRY_EVAL(ac_link) && test -s conftest${ac_exeext} 2>/dev/null; then diff --git a/configure b/configure index c4994911e224154f..3da8071a1c029507 100755 --- a/configure +++ b/configure @@ -3408,7 +3408,7 @@ if { (eval echo "$as_me:$LINENO: \"$ac_compile\"") >&5 echo "$as_me:$LINENO: \$? = $ac_status" >&5 (exit $ac_status); }; }; then for ac_declaration in \ - '' \ + '#include <stdlib.h>' \ 'extern "C" void std::exit (int) throw (); using std::exit;' \ 'extern "C" void std::exit (int); using std::exit;' \ 'extern "C" void exit (int) throw ();' \ @@ -5060,8 +5060,8 @@ main () for (i = 0; i < 256; i++) if (XOR (islower (i), ISLOWER (i)) || toupper (i) != TOUPPER (i)) - exit(2); - exit (0); + return 2; + return 0; } _ACEOF rm -f conftest$ac_exeext @@ -9369,10 +9369,6 @@ else # endif #endif -#ifdef __cplusplus -extern "C" void exit (int); -#endif - void fnord() { int i=42;} int main () { @@ -9386,7 +9382,7 @@ int main () /* dlclose (self); */ } - exit (status); + return status; } EOF if { (eval echo "$as_me:$LINENO: \"$ac_link\"") >&5 @@ -9467,10 +9463,6 @@ else # endif #endif -#ifdef __cplusplus -extern "C" void exit (int); -#endif - void fnord() { int i=42;} int main () { @@ -9484,7 +9476,7 @@ int main () /* dlclose (self); */ } - exit (status); + return status; } EOF if { (eval echo "$as_me:$LINENO: \"$ac_link\"") >&5 @@ -13163,10 +13155,6 @@ else # endif #endif -#ifdef __cplusplus -extern "C" void exit (int); -#endif - void fnord() { int i=42;} int main () { @@ -13180,7 +13168,7 @@ int main () /* dlclose (self); */ } - exit (status); + return status; } EOF if { (eval echo "$as_me:$LINENO: \"$ac_link\"") >&5 @@ -13261,10 +13249,6 @@ else # endif #endif -#ifdef __cplusplus -extern "C" void exit (int); -#endif - void fnord() { int i=42;} int main () { @@ -13278,7 +13262,7 @@ int main () /* dlclose (self); */ } - exit (status); + return status; } EOF if { (eval echo "$as_me:$LINENO: \"$ac_link\"") >&5 @@ -18926,10 +18910,6 @@ else # endif #endif -#ifdef __cplusplus -extern "C" void exit (int); -#endif - void fnord() { int i=42;} int main () { @@ -18943,7 +18923,7 @@ int main () /* dlclose (self); */ } - exit (status); + return status; } EOF if { (eval echo "$as_me:$LINENO: \"$ac_link\"") >&5 @@ -19024,10 +19004,6 @@ else # endif #endif -#ifdef __cplusplus -extern "C" void exit (int); -#endif - void fnord() { int i=42;} int main () { @@ -19041,7 +19017,7 @@ int main () /* dlclose (self); */ } - exit (status); + return status; } EOF if { (eval echo "$as_me:$LINENO: \"$ac_link\"") >&5 @@ -20546,8 +20522,8 @@ main () for (i = 0; i < 256; i++) if (XOR (islower (i), ISLOWER (i)) || toupper (i) != TOUPPER (i)) - exit(2); - exit (0); + return 2; + return 0; } _ACEOF rm -f conftest$ac_exeext @@ -24621,7 +24597,7 @@ int main() void *ret; pthread_create (&t, $defattr, func, 0); pthread_join (t, &ret); - exit (check_me != 42 || ret != &check_me); + return check_me != 42 || ret != &check_me; } _ACEOF rm -f conftest.$ac_objext conftest$ac_exeext @@ -24671,7 +24647,7 @@ int main() void *ret; pthread_create (&t, $defattr, func, 0); pthread_join (t, &ret); - exit (check_me != 42 || ret != &check_me); + return check_me != 42 || ret != &check_me; } _ACEOF rm -f conftest$ac_exeext diff --git a/configure.ac b/configure.ac index 71d577304a2e5c6f..8cc30b4483a05a48 100644 --- a/configure.ac +++ b/configure.ac @@ -173,7 +173,7 @@ int main() void *ret; pthread_create (&t, $1, func, 0); pthread_join (t, &ret); - exit (check_me != 42 || ret != &check_me); + return check_me != 42 || ret != &check_me; }]) LIBS_NOT_FOUND_1="I can't find the libraries for the thread implementation