Hello community,

here is the log from the commit of package givaro for openSUSE:Factory checked 
in at 2017-09-13 22:32:52
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/givaro (Old)
 and      /work/SRC/openSUSE:Factory/.givaro.new (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "givaro"

Wed Sep 13 22:32:52 2017 rev:5 rq:522838 version:4.0.2

Changes:
--------
--- /work/SRC/openSUSE:Factory/givaro/givaro.changes    2016-10-28 
10:46:21.000000000 +0200
+++ /work/SRC/openSUSE:Factory/.givaro.new/givaro.changes       2017-09-13 
22:32:56.297620794 +0200
@@ -1,0 +2,6 @@
+Sat Sep  9 21:59:22 UTC 2017 - jeng...@inai.de
+
+- Add 0001-pkgconfig-file-must-not-have-non-I-flags.patch.
+- Build with --disable-simd to get host-agnostic build results.
+
+-------------------------------------------------------------------

New:
----
  0001-pkgconfig-file-must-not-have-non-I-flags.patch

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

Other differences:
------------------
++++++ givaro.spec ++++++
--- /var/tmp/diff_new_pack.4gFiuC/_old  2017-09-13 22:32:56.909534652 +0200
+++ /var/tmp/diff_new_pack.4gFiuC/_new  2017-09-13 22:32:56.909534652 +0200
@@ -1,7 +1,7 @@
 #
 # spec file for package givaro
 #
-# Copyright (c) 2016 SUSE LINUX GmbH, Nuernberg, Germany.
+# Copyright (c) 2017 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
@@ -28,6 +28,7 @@
 #Git-Clone:    https://github.com/linbox-team/givaro
 Source:         https://github.com/linbox-team/givaro/archive/v%version.tar.gz
 Patch1:         givaro-doc-no-build-time.patch
+Patch2:         0001-pkgconfig-file-must-not-have-non-I-flags.patch
 # Old doxygen does not properly handle symlink recursion
 BuildRequires:  doxygen >= 1.5.7.1
 BuildRequires:  fdupes
@@ -105,12 +106,12 @@
 
 %prep
 %setup -q
-%patch -P 1 -p1
+%patch -P 1 -P 2 -p1
 
 %build
 autoreconf -fi
 %configure --disable-static --enable-doc --with-docdir="%_docdir/%name" \
-       --enable-silent-rules
+       --enable-silent-rules --disable-simd
 chmod a+x givaro-config
 make %{?_smp_mflags}
 

++++++ 0001-pkgconfig-file-must-not-have-non-I-flags.patch ++++++
>From 4775e15fe538e49237d267c677be1fe68929ab9d Mon Sep 17 00:00:00 2001
From: Jan Engelhardt <jeng...@inai.de>
Date: Sun, 10 Sep 2017 00:03:02 +0200
Subject: [PATCH] pkgconfig file must not have non -I flags

The Cflags directive in .pc files is really meant for the
preprocessor, not for the compiler. It must only contain -D and -I
flags. Otherwise, it forces flags on downstream programs that may not
want them.

In case of the build.opensuse.org build farm, it has even led to
crashes as the machine that was randomly selected to build givaro had
SSE4, emitting -msse4.1 into Cflags, while the machine randomly
selected to build fflas-ffpack had only SSE2. That cannot work.

"Cflags: -I/usr/include  -fmessage-length=0 -grecord-gcc-switches -O2
-Wall -D_FORTIFY_SOURCE=2 -fstack-protector-strong -funwind-tables
-fasynchronous-unwind-tables  -mmmx -mpopcnt -msse -msse2 -msse3
-msse4.1 -msse4.2 -msse4a -mavx -mfma -mfma4 -mbmi -mfpmath=sse
-fabi-version=6"
---
 givaro.pc.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/givaro.pc.in b/givaro.pc.in
index 274d87e..6e837ff 100644
--- a/givaro.pc.in
+++ b/givaro.pc.in
@@ -10,5 +10,5 @@ URL: http://givaro.forge.imag.fr
 Version: @VERSION@
 Requires:
 Libs: -L@libdir@ -lgivaro @LIBS@
-Cflags: -I@includedir@ @CXXFLAGS@
+Cflags: -I@includedir@
 \-------------------------------------------------------
\ No newline at end of file
-- 
2.14.1


Reply via email to