Script 'mail_helper' called by obssrc
Hello community,

here is the log from the commit of package freerdp for openSUSE:Factory checked 
in at 2021-12-06 23:59:28
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/freerdp (Old)
 and      /work/SRC/openSUSE:Factory/.freerdp.new.31177 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "freerdp"

Mon Dec  6 23:59:28 2021 rev:53 rq:935219 version:2.4.1

Changes:
--------
--- /work/SRC/openSUSE:Factory/freerdp/freerdp.changes  2021-10-25 
15:17:55.365693990 +0200
+++ /work/SRC/openSUSE:Factory/.freerdp.new.31177/freerdp.changes       
2021-12-07 00:00:40.564289851 +0100
@@ -1,0 +2,7 @@
+Thu Dec  2 15:02:32 UTC 2021 - Fabian Vogt <[email protected]>
+
+- Add patch to fix connecting without H.264-enabled FFmpeg (boo#1190823):
+  * 0001-Make-H.264-codec-optional-during-runtime.patch
+- Use %autosetup
+
+-------------------------------------------------------------------

New:
----
  0001-Make-H.264-codec-optional-during-runtime.patch

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

Other differences:
------------------
++++++ freerdp.spec ++++++
--- /var/tmp/diff_new_pack.6TWaSi/_old  2021-12-07 00:00:41.380286966 +0100
+++ /var/tmp/diff_new_pack.6TWaSi/_new  2021-12-07 00:00:41.384286951 +0100
@@ -49,6 +49,8 @@
 Source1:        freerdp-rpmlintrc
 # PATCH-FIX-UPSTREAM freerdp-channels-off-link-fix.diff -- based on 
https://github.com/FreeRDP/FreeRDP/pull/7235
 Patch0:         freerdp-builtin-channels-off-link-fix.diff
+# PATCH-FIX-UPSTREAM https://github.com/FreeRDP/FreeRDP/pull/7476
+Patch1:         0001-Make-H.264-codec-optional-during-runtime.patch
 BuildRequires:  chrpath
 BuildRequires:  cmake >= 2.8
 BuildRequires:  cups-devel
@@ -195,8 +197,7 @@
 use the uwac library.
 
 %prep
-%setup -q -n FreeRDP-%{version}
-%autopatch -p1
+%autosetup -p1 -n FreeRDP-%{version}
 
 %build
 if [ -z "$SOURCE_DATE_EPOCH" ]; then

++++++ 0001-Make-H.264-codec-optional-during-runtime.patch ++++++
>From 24b8b4e5df644ba77c673c41b90cc48c4358aaa0 Mon Sep 17 00:00:00 2001
From: Fabian Vogt <[email protected]>
Date: Thu, 2 Dec 2021 15:50:08 +0100
Subject: [PATCH] Make H.264 codec optional during runtime

It's possible that FreeRDP was built against FFmpeg, but it doesn't support
H.264. In that case, just continue without H.264 support instead of failing
hard before even trying to connect.

This is especially useful for Linux distributions which can't ship H.264
support in FFmpeg out of the box (patent issues), but allow enabling H.264
later by installing a version of FFmpeg which has it enabled.
---
 libfreerdp/core/codecs.c | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

Index: FreeRDP-2.4.1/libfreerdp/core/codecs.c
===================================================================
--- FreeRDP-2.4.1.orig/libfreerdp/core/codecs.c
+++ FreeRDP-2.4.1/libfreerdp/core/codecs.c
@@ -106,10 +106,7 @@ BOOL freerdp_client_codecs_prepare(rdpCo
 
                if (!(codecs->h264 = h264_context_new(FALSE)))
                {
-                       WLog_ERR(TAG, "Failed to create h264 codec context");
-#ifndef WITH_OPENH264_LOADING
-                       return FALSE;
-#endif
+                       WLog_WARN(TAG, "Failed to create h264 codec context");
                }
        }
 #endif

Reply via email to