Script 'mail_helper' called by obssrc Hello community, here is the log from the commit of package tre for openSUSE:Factory checked in at 2022-10-11 17:59:15 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/tre (Old) and /work/SRC/openSUSE:Factory/.tre.new.2275 (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "tre" Tue Oct 11 17:59:15 2022 rev:17 rq:1009785 version:0.8.0_git201402282055 Changes: -------- --- /work/SRC/openSUSE:Factory/tre/tre.changes 2022-01-29 20:57:28.476505200 +0100 +++ /work/SRC/openSUSE:Factory/.tre.new.2275/tre.changes 2022-10-11 17:59:21.309486172 +0200 @@ -1,0 +2,6 @@ +Tue Oct 11 09:44:20 UTC 2022 - Fabian Vogt <[email protected]> + +- Add patch to fix testsuite with bash >= 5.2: + * 0001-Remove-broken-agrep-test-entry.patch + +------------------------------------------------------------------- New: ---- 0001-Remove-broken-agrep-test-entry.patch ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ tre.spec ++++++ --- /var/tmp/diff_new_pack.bGTLw8/_old 2022-10-11 17:59:21.741486858 +0200 +++ /var/tmp/diff_new_pack.bGTLw8/_new 2022-10-11 17:59:21.745486864 +0200 @@ -33,6 +33,8 @@ # See https://github.com/laurikari/tre/pull/19. Patch1: %{name}-chicken.patch Patch2: CVE-2016-8859.patch +# https://github.com/laurikari/tre/pull/87 +Patch3: 0001-Remove-broken-agrep-test-entry.patch BuildRequires: gettext-devel BuildRequires: glibc-locale BuildRequires: libtool @@ -81,10 +83,7 @@ %lang_package %prep -%setup -q -%patch0 -p1 -%patch1 -p1 -b .chicken -%patch2 -p1 +%autosetup -p1 ./utils/autogen.sh %build ++++++ 0001-Remove-broken-agrep-test-entry.patch ++++++ >From 35f61f40d6b68928ca5d409fa9fc204ea77e2199 Mon Sep 17 00:00:00 2001 From: Fabian Vogt <[email protected]> Date: Tue, 11 Oct 2022 11:35:53 +0200 Subject: [PATCH] Remove broken agrep test entry It's meant to cause agrep to return with exit code 2, but asserts that it's exit code 1 instead. It's meant to ensure that using ".*" as pattern results in exit code 2 because it matches also an empty string. However, glob expansion results in ".*" picking up files such as "." and ".." from the CWD, which get interpreted as valid pattern. This results in exit status 1 (no match found) which is what the .ok file expects, but that's invalid. With bash 5.2, glob expansion no longer matches "." and ".." by default, so the test works as intended by accident, causing a mismatch with the expected wrong exit code. It's unfortunately not easily possible to avoid glob expansion in this case. Just remove the test for now. --- tests/agrep/exitstatus.args | 1 - tests/agrep/exitstatus.ok | 61 ------------------------------------- 2 files changed, 62 deletions(-) diff --git a/tests/agrep/exitstatus.args b/tests/agrep/exitstatus.args index 808ae77..2f53e97 100644 --- a/tests/agrep/exitstatus.args +++ b/tests/agrep/exitstatus.args @@ -5,6 +5,5 @@ this-wont-be-found . -v . # Some errors which should give exit status 2. --d .* dummy -d {1 dummy \ diff --git a/tests/agrep/exitstatus.ok b/tests/agrep/exitstatus.ok index 28427bb..bd23b4c 100644 --- a/tests/agrep/exitstatus.ok +++ b/tests/agrep/exitstatus.ok @@ -521,67 +521,6 @@ Exit status 1. Exit status 1. #### TEST: agrep -H -n -s --color --show-position -v . < exitstatus.in -Exit status 1. -#### TEST: agrep -d .* dummy exitstatus.in - -Exit status 1. -#### TEST: agrep -d .* dummy < exitstatus.in - -Exit status 1. -#### TEST: agrep -c -d .* dummy exitstatus.in -exitstatus.in:0 - -Exit status 1. -#### TEST: agrep -c -d .* dummy < exitstatus.in - -Exit status 1. -#### TEST: agrep -H -d .* dummy exitstatus.in - -Exit status 1. -#### TEST: agrep -H -d .* dummy < exitstatus.in - -Exit status 1. -#### TEST: agrep -l -d .* dummy exitstatus.in - -Exit status 1. -#### TEST: agrep -l -d .* dummy < exitstatus.in - -Exit status 1. -#### TEST: agrep -n -d .* dummy exitstatus.in - -Exit status 1. -#### TEST: agrep -n -d .* dummy < exitstatus.in - -Exit status 1. -#### TEST: agrep -s -d .* dummy exitstatus.in - -Exit status 1. -#### TEST: agrep -s -d .* dummy < exitstatus.in - -Exit status 1. -#### TEST: agrep -M -d .* dummy exitstatus.in - -Exit status 1. -#### TEST: agrep -M -d .* dummy < exitstatus.in - -Exit status 1. -#### TEST: agrep --show-position -d .* dummy exitstatus.in - -Exit status 1. -#### TEST: agrep --show-position -d .* dummy < exitstatus.in - -Exit status 1. -#### TEST: agrep --color -d .* dummy exitstatus.in - -Exit status 1. -#### TEST: agrep --color -d .* dummy < exitstatus.in - -Exit status 1. -#### TEST: agrep -H -n -s --color --show-position -d .* dummy exitstatus.in - -Exit status 1. -#### TEST: agrep -H -n -s --color --show-position -d .* dummy < exitstatus.in - Exit status 1. #### TEST: agrep -d {1 dummy exitstatus.in -- 2.36.1
