Hello community,
here is the log from the commit of package pilot-link for openSUSE:Factory checked in at 2019-10-12 22:46:18
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/pilot-link (Old)
and /work/SRC/openSUSE:Factory/.pilot-link.new.2352 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "pilot-link"
Sat Oct 12 22:46:18 2019 rev:37 rq:737662 version:0.12.5
Changes:
--------
--- /work/SRC/openSUSE:Factory/pilot-link/pilot-link.changes 2015-06-23 11:59:30.000000000 +0200
+++ /work/SRC/openSUSE:Factory/.pilot-link.new.2352/pilot-link.changes 2019-10-12 22:46:19.865230064 +0200
@@ -1,0 +2,11 @@
+Sat Oct 12 09:14:08 UTC 2019 - Dominique Leuenberger
+
+- Add pilot-link-stop-messing-with-cflags.patch: Do not mangle
+ CFLAGS in an attempt to remove -Werror. The configure check for
+ that is too simplistic and our CLAGS contains
+ -Werror=return-type. The simplystic configure 'hack' only drops
+ -Werror, resulting on =return-type staing back on the command
+ line. Since we don't add -Werror by default to CFLAGS we can
+ safely ignore this hack and drop it.
+
+-------------------------------------------------------------------
New:
----
pilot-link-stop-messing-with-cflags.patch
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Other differences:
------------------
++++++ pilot-link.spec ++++++
--- /var/tmp/diff_new_pack.He4unw/_old 2019-10-12 22:46:21.345226279 +0200
+++ /var/tmp/diff_new_pack.He4unw/_new 2019-10-12 22:46:21.345226279 +0200
@@ -1,7 +1,7 @@
#
# spec file for package pilot-link
#
-# Copyright (c) 2015 SUSE LINUX GmbH, Nuernberg, Germany.
+# Copyright (c) 2019 SUSE LINUX GmbH, Nuernberg, Germany.
#
# All modifications and additions to the file contributed by third parties
# remain the property of their copyright owners, unless otherwise agreed
@@ -12,7 +12,7 @@
# 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/
#
@@ -33,7 +33,7 @@
Obsoletes: plink < %{version}
Provides: plink = %{version}
Summary: Pilot-Link Based Synchronization Development Header Files
-License: GPL-2.0 and GPL-2.0+ and LGPL-2.1+
+License: GPL-2.0-only AND GPL-2.0-or-later AND LGPL-2.1-or-later
Group: Hardware/Palm
Source: http://downloads.pilot-link.org/%{name}-%{version}.tar.bz2
Source1: 10-usb-raw-pda.fdi
@@ -51,6 +51,8 @@
# PATCH-FIX-UPSTREAM pilot-link-0.12.5-perl514.patch idoenmez@suse.de -- Fix compilation with Perl 5.14
Patch11: pilot-link-0.12.5-perl514.patch
Patch12: pilot-link-0.12.5-udev-rules.patch
+# PATCH-FIX-OPENSUSE pilot-link-stop-messing-with-cflags.patch dimstar@opensuse.org -- Stop mangling Werror from cflags - it's not correcly done anyway
+Patch13: pilot-link-stop-messing-with-cflags.patch
BuildRoot: %{_tmppath}/%{name}-%{version}-build
%description
@@ -166,6 +168,7 @@
%patch10
%patch11
%patch12
+%patch13 -p1
# Force updating bindings/Perl/Pilot.c
rm -f bindings/Perl/Pilot.c
++++++ pilot-link-stop-messing-with-cflags.patch ++++++
Index: pilot-link-0.12.5/configure.ac
===================================================================
--- pilot-link-0.12.5.orig/configure.ac
+++ pilot-link-0.12.5/configure.ac
@@ -60,17 +60,6 @@ AC_SUBST(PISYNC_CURRENT)
AC_SUBST(PISYNC_REVISION)
AC_SUBST(PISYNC_AGE)
-dnl Eat -Werror so configure will run properly, if the user provided it
-enable_werror=no
-save_CFLAGS="$CFLAGS"
-CFLAGS=`echo $save_CFLAGS | sed -e s/-Werror//g`
-CXXFLAGS=`echo $save_CXXFLAGS | sed -e s/-Werror//g`
-if test "x$CFLAGS" != "x$save_CFLAGS"; then
- dnl -Werror was set; treat it as implicit --enable-werror below
- enable_werror="yes"
-fi
-
-
dnl Check host type
AC_CANONICAL_HOST
AC_CANONICAL_SYSTEM