Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 11 Jun 2020 01:44:21 +0200
From:      Mateusz Piotrowski <0mp@FreeBSD.org>
To:        Yuri Pankov <yuripv@yuripv.dev>, src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   Re: svn commit: r362017 - in head/usr.bin/sed: . tests
Message-ID:  <52a575ff-73aa-68a4-c3e7-5b518233ffc7@FreeBSD.org>
In-Reply-To: <a5e45631-bf32-56bc-4c03-0d0ad1d10ff2@yuripv.dev>
References:  <202006101923.05AJNwPt006602@repo.freebsd.org> <a5e45631-bf32-56bc-4c03-0d0ad1d10ff2@yuripv.dev>

next in thread | previous in thread | raw e-mail | index | archive | help
On 6/10/20 11:44 PM, Yuri Pankov wrote:
> Mateusz Piotrowski wrote:
>> Author: 0mp (doc,ports committer)
>> Date: Wed Jun 10 19:23:58 2020
>> New Revision: 362017
>> URL: https://svnweb.freebsd.org/changeset/base/362017
>>
>> Log:
>>    Read commands from stdin when -f - is passed to sed(1)
[...]
>> Modified: head/usr.bin/sed/tests/sed2_test.sh
[...]
>>   atf_init_test_cases()
>>   {
>>       atf_add_test_case inplace_command_q
>>       atf_add_test_case inplace_hardlink_src
>>       atf_add_test_case inplace_symlink_src
>>       atf_add_test_case escape_subst
>> +    atf_add_test_case commands_on_stdin
>> +    atf_add_test_case commands_on_stdin
>> +    atf_add_test_case commands_on_stdin
>>       atf_add_test_case hex_subst
>>   }
>>
>
> Am I reading it wrong, or is it the same test case added 3 times?

Sigh, yes, it does not make any sense. Thanks for spotting.

Removing those 2 extra lines should do, right? Can I commit it?




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?52a575ff-73aa-68a4-c3e7-5b518233ffc7>