Hello community, here is the log from the commit of package php7-imagick for openSUSE:Factory checked in at 2018-05-11 09:14:27 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/php7-imagick (Old) and /work/SRC/openSUSE:Factory/.php7-imagick.new (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "php7-imagick" Fri May 11 09:14:27 2018 rev:6 rq:605656 version:3.4.3 Changes: -------- --- /work/SRC/openSUSE:Factory/php7-imagick/php7-imagick.changes 2018-03-19 23:38:17.993522875 +0100 +++ /work/SRC/openSUSE:Factory/.php7-imagick.new/php7-imagick.changes 2018-05-11 09:14:32.650625361 +0200 @@ -1,0 +2,24 @@ +Wed May 9 06:37:45 UTC 2018 - jweberho...@weberhofer.at + +- removed Test 229 which fails occasionally with timeouts + on the OBS build server + +------------------------------------------------------------------- +Mon May 7 06:17:40 UTC 2018 - jweberho...@weberhofer.at + +- Print test failures in detail + +------------------------------------------------------------------- +Sun May 6 10:09:31 UTC 2018 - jweberho...@weberhofer.at + +Fixes for bnc#1091929 + +- Due to a packaging change in ghostscript the ghostscript-fonts-std + has to be included explicitly to successfully run the tests. Elsewise + the required helvetica-font is missing. + +- ImageMagick's commit 97a319 makes ImagaMagick to no longer throw an + an exception if image registry tag is not found. + Added imagic-3.4.3-relax_test_150_to_follow_upstream_change.patch + +------------------------------------------------------------------- New: ---- imagic-3.4.3-relax_test_150_to_follow_upstream_change.patch ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ php7-imagick.spec ++++++ --- /var/tmp/diff_new_pack.BYYmWE/_old 2018-05-11 09:14:34.550556534 +0200 +++ /var/tmp/diff_new_pack.BYYmWE/_new 2018-05-11 09:14:34.554556389 +0200 @@ -28,8 +28,11 @@ Source0: https://pecl.php.net/get/%{pkg_name}-%{version}.tgz Source1: %{pkg_name}.ini Patch0: imagick-reproducible.patch +#PATCH-FIX-UPSTREAM test_150_to_follow_upstream_change.patch bnc#1091929 +Patch1: imagic-3.4.3-relax_test_150_to_follow_upstream_change.patch BuildRequires: %{php_name}-devel >= 7.0.1 BuildRequires: ImageMagick-devel >= 6.5.3.10 +BuildRequires: ghostscript-fonts-std BuildRequires: re2c Provides: php-%{pkg_name} = %{version} Obsoletes: php-%{pkg_name} < %{version} @@ -48,7 +51,10 @@ %prep %setup -q -n %{pkg_name}-%{version} %patch0 -p1 +%patch1 -p1 mkdir %{name} +# Ignore know failed test on OBS with timeout +rm tests/229_Tutorial_fxAnalyzeImage_case1.phpt %build %{_bindir}/phpize @@ -57,7 +63,8 @@ make %{?_smp_mflags} %check -make %{?_smp_mflags} PHP_EXECUTABLE=%{__php} NO_INTERACTION=1 test +make %{?_smp_mflags} PHP_EXECUTABLE=%{__php} NO_INTERACTION=1 test \ + || { for f in tests/*.out; do cat $f; echo '------'; done; exit 1; } %install make DESTDIR=%{buildroot} install INSTALL_ROOT=%{buildroot} ++++++ imagic-3.4.3-relax_test_150_to_follow_upstream_change.patch ++++++ >From a516b549693ea373c0e4cb0286337f2e0fcd7f3c Mon Sep 17 00:00:00 2001 From: Remi Collet <r...@remirepo.net> Date: Thu, 14 Dec 2017 12:58:28 +0100 Subject: [PATCH] Fix #220 relax test to follow upstream change --- tests/150_Imagick_setregistry.phpt | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/tests/150_Imagick_setregistry.phpt b/tests/150_Imagick_setregistry.phpt index eb2b055..1b96452 100644 --- a/tests/150_Imagick_setregistry.phpt +++ b/tests/150_Imagick_setregistry.phpt @@ -35,10 +35,10 @@ if (array_key_exists("temporary-path", $registry) == true) { try { $tmpPath = Imagick::getRegistry("non-existent string"); - echo "Expected exception not thrown.\n"; + var_dump($tmpPath == false); // Since 6.9.9-26, no exception raised } catch (\ImagickException $ie) { - echo $ie->getMessage() . "\n"; + var_dump(true); } @@ -47,4 +47,4 @@ catch (\ImagickException $ie) { Temporary-path was empty at start. Temporary path was set correctly. Temporary path was listed correctly. -Imagick::getRegistry exception (%s) +bool(true)