Script 'mail_helper' called by obssrc
Hello community,

here is the log from the commit of package t38modem for openSUSE:Factory 
checked in at 2022-04-10 19:05:32
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/t38modem (Old)
 and      /work/SRC/openSUSE:Factory/.t38modem.new.1900 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "t38modem"

Sun Apr 10 19:05:32 2022 rev:20 rq:968026 version:4.6.0

Changes:
--------
--- /work/SRC/openSUSE:Factory/t38modem/t38modem.changes        2015-09-13 
09:45:03.000000000 +0200
+++ /work/SRC/openSUSE:Factory/.t38modem.new.1900/t38modem.changes      
2022-04-10 19:05:50.766206230 +0200
@@ -1,0 +2,8 @@
+Sat Apr  2 13:55:15 UTC 2022 - Jan Engelhardt <jeng...@inai.de>
+
+- Update to release 4.6.0
+  * No changelog was provided
+- Remove undef.diff
+- Add 0001-build-resolve-compiler-warnings-and-errors-with-ptli.patch
+
+-------------------------------------------------------------------

Old:
----
  undef.diff
  v3.13.0.tar.gz

New:
----
  0001-build-resolve-compiler-warnings-and-errors-with-ptli.patch
  v4.6.0.tar.gz

++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Other differences:
------------------
++++++ t38modem.spec ++++++
--- /var/tmp/diff_new_pack.lvMMJZ/_old  2022-04-10 19:05:51.382199410 +0200
+++ /var/tmp/diff_new_pack.lvMMJZ/_new  2022-04-10 19:05:51.390199322 +0200
@@ -1,7 +1,7 @@
 #
 # spec file for package t38modem
 #
-# Copyright (c) 2015 SUSE LINUX GmbH, Nuernberg, Germany.
+# Copyright (c) 2022 SUSE LLC
 #
 # All modifications and additions to the file contributed by third parties
 # remain the property of their copyright owners, unless otherwise agreed
@@ -12,24 +12,25 @@
 # license that conforms to the Open Source Definition (Version 1.9)
 # published by the Open Source Initiative.
 
-# Please submit bugfixes or comments via http://bugs.opensuse.org/
+# Please submit bugfixes or comments via https://bugs.opensuse.org/
 #
 
 
 Name:           t38modem
-Version:        3.13.0
+Version:        4.6.0
 Release:        0
 Summary:        T.38 for OpenH323 (modem-compatible interface for IP telephony)
 License:        MPL-1.1
 Group:          Productivity/Telephony/H323/Servers
-Url:            http://www.openh323.org
+URL:            https://github.com/T38Modem/t38modem
 
-Source:         
https://github.com/PeteDavidson/t38modem/archive/v%version.tar.gz
-Patch1:         undef.diff
-BuildRoot:      %{_tmppath}/%{name}-%{version}-build
+Source:         https://github.com/T38Modem/t38modem/archive/v%version.tar.gz
+Patch1:         0001-build-resolve-compiler-warnings-and-errors-with-ptli.patch
 BuildRequires:  gcc-c++
 BuildRequires:  pkg-config
-BuildRequires:  pkgconfig(opal) >= 3.10
+BuildRequires:  pkgconfig(libsrtp2)
+BuildRequires:  pkgconfig(opal) >= 3.18.6
+BuildRequires:  pkgconfig(ptlib) >= 2.18.6
 
 %description
 This package contains a modem interface for IP telephony using the
@@ -37,8 +38,7 @@
 which resembles a FAX modem. A sample HylaFAX setup is also provided.
 
 %prep
-%setup -q
-%patch -P 1 -p1
+%autosetup -p1
 
 %build
 make CFLAGS="%optflags" CXXFLAGS="%optflags" %{?_smp_mflags}
@@ -49,7 +49,6 @@
 cp -a t38modem "$b/%_sbindir/"
 
 %files
-%defattr(-,root,root)
 %doc ReadMe.txt
 %doc HylaFAX/config.ttyx
 %_sbindir/t38modem

++++++ 0001-build-resolve-compiler-warnings-and-errors-with-ptli.patch ++++++
>From 98786058110416c18096fba9945d7a86917580c2 Mon Sep 17 00:00:00 2001
From: Jan Engelhardt <jeng...@inai.de>
Date: Sat, 2 Apr 2022 16:04:40 +0200
Subject: [PATCH] build: resolve compiler warnings and errors with ptlib-2.18.8
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
References: https://github.com/T38Modem/t38modem/pull/2

The definition of PINDEX changed in ptlib commit
6629d94fcb3cfd369a9629399a20a217429b4d60 .

dle.cxx: In member function ???int DLEData::GetDleData(void*, PINDEX)???:
dle.cxx:180:13: error: narrowing conversion of ???-1??? from ???int??? to
???long unsigned int??? [-Wnarrowing]
  180 |       case -1:
pmutils.cxx: In member function ???int ChunkStream::write(const void*, 
PINDEX)???:
pmutils.cxx:129:11: warning: comparison of integer expressions of
different signedness: ???int??? and ???PINDEX??? {aka ???long unsigned int???} 
[-Wsign-compare]
  129 |   if (len > count)
pmutils.cxx: In member function ???int ChunkStream::read(void*, PINDEX)???:
pmutils.cxx:145:11: warning: comparison of integer expressions of
different signedness: ???int??? and ???PINDEX??? {aka ???long unsigned int???} 
[-Wsign-compare]
  145 |   if (len > count)
---
 dle.cxx     | 3 ++-
 pmutils.cxx | 4 ++--
 2 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/dle.cxx b/dle.cxx
index dc357d6..43aa795 100644
--- a/dle.cxx
+++ b/dle.cxx
@@ -176,7 +176,8 @@ int DLEData::GetDleData(void *pBuf, PINDEX count)
     if (cGet > (PINDEX)sizeof(tmp))
       cGet = sizeof(tmp);
 
-    switch( cGet = GetData(tmp, cGet) ) {
+    int ret = GetData(tmp, cGet);
+    switch (ret) {
       case -1:
         *p++ = DLE;
         *p++ = ETX;
diff --git a/pmutils.cxx b/pmutils.cxx
index 8437612..46e90e8 100644
--- a/pmutils.cxx
+++ b/pmutils.cxx
@@ -121,7 +121,7 @@ void ModemThreadChild::SignalStop()
 ///////////////////////////////////////////////////////////////
 int ChunkStream::write(const void *pBuf, PINDEX count)
 {
-  int len = sizeof(data) - last;
+  size_t len = sizeof(data) - last;
 
   if (!len)
     return -1;
@@ -142,7 +142,7 @@ int ChunkStream::read(void *pBuf, PINDEX count)
 
   int len = last - first;
 
-  if (len > count)
+  if (len > 0 && static_cast<size_t>(len) > count)
     len = count;
 
   memcpy(pBuf, data + first, len);
-- 
2.35.1


++++++ v3.13.0.tar.gz -> v4.6.0.tar.gz ++++++
++++ 13216 lines of diff (skipped)

Reply via email to