From d62552e412a23d26e6044e75132fb973a48a4125 Mon Sep 17 00:00:00 2001 From: Remi Collet Date: Fri, 10 Apr 2020 15:55:29 +0200 Subject: add upstream patch for test suite and PHP 8 --- php-pecl-rpminfo.spec | 9 +++- rpminfo-tests.patch | 143 ++++++++++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 151 insertions(+), 1 deletion(-) create mode 100644 rpminfo-tests.patch diff --git a/php-pecl-rpminfo.spec b/php-pecl-rpminfo.spec index 450a959..2819752 100644 --- a/php-pecl-rpminfo.spec +++ b/php-pecl-rpminfo.spec @@ -23,11 +23,14 @@ Summary: RPM information Name: %{?sub_prefix}php-pecl-%{pecl_name} Version: 0.5.0 -Release: 1%{?dist}%{!?nophptag:%(%{__php} -r 'echo ".".PHP_MAJOR_VERSION.".".PHP_MINOR_VERSION;')} +Release: 2%{?dist}%{!?nophptag:%(%{__php} -r 'echo ".".PHP_MAJOR_VERSION.".".PHP_MINOR_VERSION;')} License: PHP URL: https://pecl.php.net/package/%{pecl_name} Source0: https://pecl.php.net/get/%{pecl_name}-%{version}.tgz +# Upstream patch for test suite +Patch0: %{pecl_name}-tests.patch + BuildRequires: %{?dtsprefix}gcc BuildRequires: pkgconfig(rpm) >= 4.11.3 BuildRequires: %{?scl_prefix}php-devel > 7 @@ -89,6 +92,7 @@ sed -e 's/role="test"/role="src"/' \ mv %{pecl_name}-%{version} NTS cd NTS +%patch0 -p1 -b .up # Sanity check, really often broken extver=$(sed -n '/#define PHP_RPMINFO_VERSION/{s/.* "//;s/".*$//;p}' php_rpminfo.h) @@ -222,6 +226,9 @@ REPORT_EXIT_STATUS=1 \ %changelog +* Fri Apr 10 2020 Remi Collet - 0.5.0-2 +- add upstream patch for test suite and PHP 8 + * Tue Apr 7 2020 Remi Collet - 0.5.0-1 - update to 0.5.0 diff --git a/rpminfo-tests.patch b/rpminfo-tests.patch new file mode 100644 index 0000000..fedb9bd --- /dev/null +++ b/rpminfo-tests.patch @@ -0,0 +1,143 @@ +From 2aa9dc045d03c3cf55eefa769b4a54b20c41bc18 Mon Sep 17 00:00:00 2001 +From: Remi Collet +Date: Fri, 10 Apr 2020 15:49:35 +0200 +Subject: split tests for PHP 7/8 + +--- + package.xml | 4 +++- + tests/002-rpmvercmp.phpt | 13 ------------- + tests/010-rpmvercmp_error7.phpt | 25 +++++++++++++++++++++++++ + tests/011-rpmvercmp_error8.phpt | 31 +++++++++++++++++++++++++++++++ + 4 files changed, 59 insertions(+), 14 deletions(-) + create mode 100644 tests/010-rpmvercmp_error7.phpt + create mode 100644 tests/011-rpmvercmp_error8.phpt + +diff --git a/tests/002-rpmvercmp.phpt b/tests/002-rpmvercmp.phpt +index 7da40b4..d83cf67 100644 +--- a/tests/002-rpmvercmp.phpt ++++ b/tests/002-rpmvercmp.phpt +@@ -35,21 +35,8 @@ foreach ($cases as $case) { + } + } + if ($ok) echo "OK\n"; +-// Errors +-var_dump(rpmvercmp()); +-var_dump(rpmvercmp("a")); +-var_dump(rpmvercmp("a", "b", "c")); + ?> + Done + --EXPECTF-- + OK +- +-Warning: rpmvercmp() expects exactly 2 parameters, 0 given in %s/002-rpmvercmp.php on line %d +-NULL +- +-Warning: rpmvercmp() expects exactly 2 parameters, 1 given in %s/002-rpmvercmp.php on line %d +-NULL +- +-Warning: rpmvercmp() expects exactly 2 parameters, 3 given in %s/002-rpmvercmp.php on line %d +-NULL + Done +diff --git a/tests/010-rpmvercmp_error7.phpt b/tests/010-rpmvercmp_error7.phpt +new file mode 100644 +index 0000000..98bfeb3 +--- /dev/null ++++ b/tests/010-rpmvercmp_error7.phpt +@@ -0,0 +1,25 @@ ++--TEST-- ++Check for rpmvercmp function error ++--SKIPIF-- ++= 80000) print "skip only for PHP 7"; ++?> ++--FILE-- ++ ++Done ++--EXPECTF-- ++ ++Warning: rpmvercmp() expects exactly 2 parameters, 0 given in %s/002-rpmvercmp.php on line %d ++NULL ++ ++Warning: rpmvercmp() expects exactly 2 parameters, 1 given in %s/002-rpmvercmp.php on line %d ++NULL ++ ++Warning: rpmvercmp() expects exactly 2 parameters, 3 given in %s/002-rpmvercmp.php on line %d ++NULL ++Done +diff --git a/tests/011-rpmvercmp_error8.phpt b/tests/011-rpmvercmp_error8.phpt +new file mode 100644 +index 0000000..3db3564 +--- /dev/null ++++ b/tests/011-rpmvercmp_error8.phpt +@@ -0,0 +1,31 @@ ++--TEST-- ++Check for rpmvercmp function ++--SKIPIF-- ++ ++--FILE-- ++getMessage(), "\n"; ++} ++try { ++ var_dump(rpmvercmp("a")); ++} catch (ArgumentCountError $e) { ++ echo $e->getMessage(), "\n"; ++} ++try { ++ var_dump(rpmvercmp("a", "b", "c")); ++} catch (ArgumentCountError $e) { ++ echo $e->getMessage(), "\n"; ++} ++?> ++Done ++--EXPECTF-- ++rpmvercmp() expects exactly 2 parameters, 0 given ++rpmvercmp() expects exactly 2 parameters, 1 given ++rpmvercmp() expects exactly 2 parameters, 3 given ++Done +-- +cgit v1.1 + +From 895bc3dbd5dd48c97561bb0758f86a8c67bd6874 Mon Sep 17 00:00:00 2001 +From: Remi Collet +Date: Fri, 10 Apr 2020 15:53:11 +0200 +Subject: fix test + +--- + tests/010-rpmvercmp_error7.phpt | 6 +++--- + 1 file changed, 3 insertions(+), 3 deletions(-) + +diff --git a/tests/010-rpmvercmp_error7.phpt b/tests/010-rpmvercmp_error7.phpt +index 98bfeb3..c9b3875 100644 +--- a/tests/010-rpmvercmp_error7.phpt ++++ b/tests/010-rpmvercmp_error7.phpt +@@ -14,12 +14,12 @@ var_dump(rpmvercmp("a", "b", "c")); + Done + --EXPECTF-- + +-Warning: rpmvercmp() expects exactly 2 parameters, 0 given in %s/002-rpmvercmp.php on line %d ++Warning: rpmvercmp() expects exactly 2 parameters, 0 given in %s + NULL + +-Warning: rpmvercmp() expects exactly 2 parameters, 1 given in %s/002-rpmvercmp.php on line %d ++Warning: rpmvercmp() expects exactly 2 parameters, 1 given in %s + NULL + +-Warning: rpmvercmp() expects exactly 2 parameters, 3 given in %s/002-rpmvercmp.php on line %d ++Warning: rpmvercmp() expects exactly 2 parameters, 3 given in %s + NULL + Done +-- +cgit v1.1 + -- cgit