From f6ddf8d13c35075b630ab77cc5cbc8c57e8d5608 Mon Sep 17 00:00:00 2001 From: Adam Williamson Date: Apr 22 2016 22:44:47 +0000 Subject: backport proposed patch to build with latest freerdp (BGO #765444) --- diff --git a/0001-handle-new-freerdp-pkgconfig-name.patch b/0001-handle-new-freerdp-pkgconfig-name.patch new file mode 100644 index 0000000..4f88ab0 --- /dev/null +++ b/0001-handle-new-freerdp-pkgconfig-name.patch @@ -0,0 +1,39 @@ +From 8d072483ffff3a4e752c35811fb562f61d206f68 Mon Sep 17 00:00:00 2001 +From: Adam Williamson +Date: Fri, 22 Apr 2016 14:54:09 -0700 +Subject: [PATCH] handle new freerdp pkgconfig name + +freerdp has now changed its pkgconfig name to 'freerdp2' - +https://github.com/FreeRDP/FreeRDP/commit/6fa36081 . Assuming +we can build against both 1 and 2, we should handle both names. +--- + configure.ac | 6 +++++- + 1 file changed, 5 insertions(+), 1 deletion(-) + +diff --git a/configure.ac b/configure.ac +index 585c016..362951d 100644 +--- a/configure.ac ++++ b/configure.ac +@@ -59,6 +59,7 @@ AM_CONDITIONAL([VINAGRE_ENABLE_SSH], [test "x$have_ssh" = "xyes"]) + + # Whether to enable support for RDP. + RDP_DEPS="freerdp x11" ++RDP_2_DEPS="freerdp2 x11" + AC_ARG_ENABLE([rdp], + [AS_HELP_STRING([--disable-rdp], + [Disable Remote Desktop Protocol (RDP) support])]) +@@ -68,7 +69,10 @@ AS_IF([test "x$enable_rdp" != "xno"], + [have_rdp=yes + PKG_CHECK_EXISTS(freerdp >= 1.1, + [AC_DEFINE([HAVE_FREERDP_1_1], [1], [FreeRDP is of version 1.1 or newer])], [])], +- [have_rdp=no])], ++ [PKG_CHECK_EXISTS([$RDP_2_DEPS], ++ [have_rdp=yes ++ RDP_DEPS=$RDP_2_DEPS ++ AC_DEFINE([HAVE_FREERDP_1_1], [1], [FreeRDP is of version 1.1 or newer])], [have_rdp=no])])], + [have_rdp=no]) + + AS_IF([test "x$have_rdp" = "xyes"], +-- +2.7.3 + diff --git a/vinagre.spec b/vinagre.spec index 34a0dea..f5ed098 100644 --- a/vinagre.spec +++ b/vinagre.spec @@ -12,6 +12,8 @@ License: GPLv2+ URL: https://wiki.gnome.org/Apps/Vinagre #VCS: git:git://git.gnome.org/vinagre Source0: https://download.gnome.org/sources/%{name}/3.20/%{name}-%{version}.tar.xz +# FIXME: bug link +Patch0: 0001-handle-new-freerdp-pkgconfig-name.patch %if 0%{?with_spice} BuildRequires: pkgconfig(spice-client-gtk-3.0) @@ -31,6 +33,11 @@ BuildRequires: /usr/bin/appstream-util BuildRequires: intltool BuildRequires: itstool BuildRequires: vala-devel +# For Patch0 autoreconf +BuildRequires: gnome-common +BuildRequires: libappstream-glib-devel +BuildRequires: autoconf +BuildRequires: automake # for /usr/share/dbus-1/services Requires: dbus @@ -59,10 +66,13 @@ Apart from the VNC protocol, vinagre supports Spice and RDP. %prep -%setup -q +%autosetup %build +# copied from autogen.sh, needed for Patch0 +ACLOCAL_FLAGS="$ACLOCAL_FLAGS" USE_GNOME2_MACROS=1 . gnome-autogen.sh + %configure \ %if 0%{?with_spice} --enable-spice \ @@ -119,6 +129,7 @@ gtk-update-icon-cache %{_datadir}/icons/hicolor &>/dev/null || : %changelog * Fri Apr 22 2016 Adam Williamson - 3.20.1-2 - rebuild for changed freerdp sonames +- backport patch to handle freerdp pkgconfig module rename * Mon Apr 11 2016 David King - 3.20.1-1 - Update to 3.20.1