Script 'mail_helper' called by obssrc Hello community, here is the log from the commit of package OpenIPMI for openSUSE:Factory checked in at 2024-08-13 13:22:13 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/OpenIPMI (Old) and /work/SRC/openSUSE:Factory/.OpenIPMI.new.7232 (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Package is "OpenIPMI" Tue Aug 13 13:22:13 2024 rev:56 rq:1193362 version:2.0.32.3+git.05faa36e Changes: -------- --- /work/SRC/openSUSE:Factory/OpenIPMI/OpenIPMI.changes 2022-05-12 22:57:44.192585433 +0200 +++ /work/SRC/openSUSE:Factory/.OpenIPMI.new.7232/OpenIPMI.changes 2024-08-13 13:22:22.904090758 +0200 @@ -1,0 +2,26 @@ +Tue Jun 18 12:31:39 UTC 2024 - Martin Jambor <mjambor@suse.com> + +- Add OpenIPMI-c99.patch in order to make the package buildable with + GCC 14. Patch comes from upstream + https://sourceforge.net/p/openipmi/patches/38/ [[boo#1225871]] + +------------------------------------------------------------------- +Tue May 10 13:34:05 UTC 2022 - trenn@suse.de + +- Update to version 2.0.32.3+git.05faa36e: + * entity: Handle NULL entity in internal_fru_fetch_done() + * entity: Keep the entity around duringa FRU fetch + * Move to version 2.0.32 + * Fix readline and python support + * sol: Fix multiple flush queuing issue +- Patches that are mainline: +D fix-conversions.patch +D unix_thread-Remove-the-fd-handler-sets-before-it-s-set-up.patch +D lanserv-Add-the-judgment-on-the-validity-of-length-in-emu_cmd.c-and-session-in-lanserv_ipmi.c.patch +D OpenIPMI-no-return-in-nonvoid-function.patch +- Refreshed patches not yet mainline: +M openipmi-tinfo.patch +M fix_dia_version_detection.patch + + +------------------------------------------------------------------- Old: ---- OpenIPMI-2.0.31.tar.gz OpenIPMI-no-return-in-nonvoid-function.patch fix-conversions.patch lanserv-Add-the-judgment-on-the-validity-of-length-in-emu_cmd.c-and-session-in-lanserv_ipmi.c.patch unix_thread-Remove-the-fd-handler-sets-before-it-s-set-up.patch New: ---- OpenIPMI-c99.patch _service _servicedata openipmi-2.0.32.3+git.05faa36e.obscpio openipmi.obsinfo BETA DEBUG BEGIN: Old:D lanserv-Add-the-judgment-on-the-validity-of-length-in-emu_cmd.c-and-session-in-lanserv_ipmi.c.patch D OpenIPMI-no-return-in-nonvoid-function.patch - Refreshed patches not yet mainline: Old:- Patches that are mainline: D fix-conversions.patch D unix_thread-Remove-the-fd-handler-sets-before-it-s-set-up.patch Old:D unix_thread-Remove-the-fd-handler-sets-before-it-s-set-up.patch D lanserv-Add-the-judgment-on-the-validity-of-length-in-emu_cmd.c-and-session-in-lanserv_ipmi.c.patch D OpenIPMI-no-return-in-nonvoid-function.patch Old:D fix-conversions.patch D unix_thread-Remove-the-fd-handler-sets-before-it-s-set-up.patch D lanserv-Add-the-judgment-on-the-validity-of-length-in-emu_cmd.c-and-session-in-lanserv_ipmi.c.patch BETA DEBUG END: BETA DEBUG BEGIN: New: - Add OpenIPMI-c99.patch in order to make the package buildable with GCC 14. Patch comes from upstream BETA DEBUG END: ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ OpenIPMI.spec ++++++ --- /var/tmp/diff_new_pack.Icw4XP/_old 2024-08-13 13:22:24.120141426 +0200 +++ /var/tmp/diff_new_pack.Icw4XP/_new 2024-08-13 13:22:24.124141592 +0200 @@ -31,37 +31,30 @@ %endif Name: OpenIPMI -Version: 2.0.31 +Version: 2.0.32.3+git.05faa36e Release: 0 Summary: Service processor access via IPMI License: LGPL-2.1-or-later Group: System/Monitoring URL: http://openipmi.sourceforge.net -Source0: http://prdownloads.sourceforge.net/openipmi/%{name}-%{version}.tar.gz +Source0: openipmi-%{version}.tar.gz Source1: sysconfig.ipmi Source2: ipmi.service Source3: openipmi-helper Source4: bootstrap Source99: OpenIPMI-rpmlintrc -# Patch0: OpenIPMI-2.0.25..HEAD_git.diff Patch1: OpenIPMI-prefer_perl_vendor.patch -Patch2: fix-conversions.patch -# PATCH-FIX-UPSTREAM do not use bundled libedit -# Patch3: 0003-Apply-OpenIPMI-2.0.21-nobundle.patch.patch -# PATCH-FIX-UPSTREAM detect tinfo from ncurses properly -Patch4: openipmi-tinfo.patch +Patch2: openipmi-tinfo.patch # link with ncurses -Patch5: OpenIMPI-add-libncurses.patch -# some int functions do not return a value -Patch6: OpenIPMI-no-return-in-nonvoid-function.patch -Patch7: fix_dia_version_detection.patch -Patch8: use_python3_shebang -Patch9: lanserv-Add-the-judgment-on-the-validity-of-length-in-emu_cmd.c-and-session-in-lanserv_ipmi.c.patch -Patch10: unix_thread-Remove-the-fd-handler-sets-before-it-s-set-up.patch +Patch3: OpenIMPI-add-libncurses.patch +Patch4: fix_dia_version_detection.patch +Patch5: use_python3_shebang # PATCH-FIX-UPSTREAM OpenIPMI-autoconf-m4-ax-python-devel-serial25.patch # replace autoconf script serial18 addtion by serial 25 capable of finding python3.10 # retains upstream change https://sourceforge.net/p/openipmi/code/ci/53d2b36b58383d155e2410cae3b30975a... -Patch11: OpenIPMI-autoconf-m4-ax-python-devel-serial25.patch +Patch6: OpenIPMI-autoconf-m4-ax-python-devel-serial25.patch +# Backport of upstream https://sourceforge.net/p/openipmi/patches/38/ to fix boo#1225871 +Patch7: OpenIPMI-c99.patch BuildRequires: autoconf-archive BuildRequires: gd-devel @@ -152,7 +145,7 @@ that makes use of it. %prep -%autosetup -p1 +%autosetup -n openipmi-%{version} -p1 rm -rf ./libedit ++++++ OpenIPMI-c99.patch ++++++ C type errors in the SWIG-generated Perl bindings The first change fixes an error with newer compilers: OpenIPMI_wrap.c: In function ‘_wrap_strconstarray_val_set’: OpenIPMI_wrap.c:10491:27: error: assignment to ‘const char **’ from incompatible pointer type ‘char **’ 10491 | if (arg1) (arg1)->val = arg2; | ^ The second change is also about a compiler error: In file included from /usr/lib64/perl5/CORE/perl.h:4530, from OpenIPMI_wrap.c:751: OpenIPMI_wrap.c: In function ‘_wrap_ipmi_sol_conn_t_write’: /usr/lib64/perl5/CORE/sv.h:1952:31: error: passing argument 3 of ‘Perl_SvPV_helper’ from incompatible pointer type 1952 | Perl_SvPV_helper(aTHX_ sv, &len, flags, SvPVnormal_type_, \ /usr/lib64/perl5/CORE/sv.h:1972:37: note: in expansion of macro ‘SvPV_flags’ 1972 | #define SvPV(sv, len) SvPV_flags(sv, len, SV_GMAGIC) | ^~~~~~~~~~ OpenIPMI_wrap.c:27664:24: note: in expansion of macro ‘SvPV’ 27664 | (&arg2)->val = SvPV(tempsv, (&arg2)->len); | ^~~~ In file included from /usr/lib64/perl5/CORE/perl.h:7812: /usr/lib64/perl5/CORE/sv_inline.h:908:33: note: expected ‘STRLEN * const’ {aka ‘long unsigned int * const’} but argument is of type ‘int *’ 908 | STRLEN * const lp, | ~~~~~~~~~~~~~~~^~ But the existing code looks broken on big-endian 64-bit architectures, too. diff --git a/swig/OpenIPMI.i b/swig/OpenIPMI.i index 8e674a94e6f85f62..db2aa1b5292a276f 100644 --- a/swig/OpenIPMI.i +++ b/swig/OpenIPMI.i @@ -359,7 +359,7 @@ typedef struct iargarray %} typedef struct strconstarray { - char **val; + const char **val; int len; } strconstarray; typedef struct argarray diff --git a/swig/perl/OpenIPMI_lang.i b/swig/perl/OpenIPMI_lang.i index d13b62f09e69e02a..d7482d1135cdc3e4 100644 --- a/swig/perl/OpenIPMI_lang.i +++ b/swig/perl/OpenIPMI_lang.i @@ -292,7 +292,9 @@ $1.val = NULL; $1.len = 0; } else { - $1.val = SvPV(tempsv, $1.len); + STRLEN len; + $1.val = SvPV(tempsv, len); + $1.len = len; } } ++++++ _service ++++++ <services> <service name="obs_scm" mode="localonly"> <param name="url">https://github.com/cminyard/openipmi.git</param> <param name="scm">git</param> <param name="versionrewrite-pattern">v(.*)</param> <param name="versionformat">@PARENT_TAG@.@TAG_OFFSET@+git.%h</param> <param name="changesgenerate">enable</param> </service> <service name="set_version" mode="localonly"/> <service name="tar" mode="buildtime"/> <service name="recompress" mode="buildtime"> <param name="file">*.tar</param> <param name="compression">gz</param> </service> </services> ++++++ _servicedata ++++++ <servicedata> <service name="tar_scm"> <param name="url">https://github.com/cminyard/openipmi.git</param> <param name="changesrevision">05faa36e93eaf7e150bd43d7935e755b6e8ca617</param></service></servicedata> (No newline at EOF) ++++++ fix_dia_version_detection.patch ++++++ --- /var/tmp/diff_new_pack.Icw4XP/_old 2024-08-13 13:22:24.244146592 +0200 +++ /var/tmp/diff_new_pack.Icw4XP/_new 2024-08-13 13:22:24.244146592 +0200 @@ -1,8 +1,8 @@ -Index: OpenIPMI-2.0.31/configure.ac +Index: openipmi-2.0.32.3+git.05faa36e/configure.ac =================================================================== ---- OpenIPMI-2.0.31.orig/configure.ac -+++ OpenIPMI-2.0.31/configure.ac -@@ -720,11 +720,14 @@ AC_HAVE_FUNCS(syslog) +--- openipmi-2.0.32.3+git.05faa36e.orig/configure.ac ++++ openipmi-2.0.32.3+git.05faa36e/configure.ac +@@ -729,11 +729,14 @@ AC_HAVE_FUNCS(syslog) DIA= AC_PATH_PROG(diaprog, dia) if test "x$diaprog" != "x"; then ++++++ openipmi-tinfo.patch ++++++ --- /var/tmp/diff_new_pack.Icw4XP/_old 2024-08-13 13:22:24.280148092 +0200 +++ /var/tmp/diff_new_pack.Icw4XP/_new 2024-08-13 13:22:24.284148258 +0200 @@ -1,8 +1,8 @@ -Index: OpenIPMI-2.0.31/configure.ac +Index: openipmi-2.0.32.3+git.05faa36e/configure.ac =================================================================== ---- OpenIPMI-2.0.31.orig/configure.ac -+++ OpenIPMI-2.0.31/configure.ac -@@ -768,10 +768,12 @@ AX_CONFIG_FEATURE( +--- openipmi-2.0.32.3+git.05faa36e.orig/configure.ac ++++ openipmi-2.0.32.3+git.05faa36e/configure.ac +@@ -777,10 +777,12 @@ AX_CONFIG_FEATURE( [epoll_pwait], [This platform supports epoll(7) with epoll_pwait(2)], [HAVE_EPOLL_PWAIT], [This platform supports epoll(7) with epoll_pwait(2).]) ++++++ openipmi.obsinfo ++++++ name: openipmi version: 2.0.32.3+git.05faa36e mtime: 1649625713 commit: 05faa36e93eaf7e150bd43d7935e755b6e8ca617