Re: svn commit: r316536 - in head: contrib/netbsd-tests/usr.bin/grep usr.bin/grep/tests

2017-04-05 Thread Kyle Evans
On Wed, Apr 5, 2017 at 6:06 PM, Ngie Cooper (yaneurabeya) < yaneurab...@gmail.com> wrote: > > > > > > Yup! > -Ngie > > Given no objections from emaste@ (seems unlikely, given the content), can one of you please commit this when able? It seems like a good idea for the sake of providing context fo

Re: svn commit: r316536 - in head: contrib/netbsd-tests/usr.bin/grep usr.bin/grep/tests

2017-04-05 Thread Ngie Cooper (yaneurabeya)
> On Apr 5, 2017, at 12:33, Kyle Evans wrote: > > On Wed, Apr 5, 2017 at 1:48 PM, Ngie Cooper (yaneurabeya) > wrote: > > As a general rule of thumb, if it doesn’t pass, it needs to be marked > as an expected failure until it passes. Not really a big deal right now > (because ci.free

Re: svn commit: r316536 - in head: contrib/netbsd-tests/usr.bin/grep usr.bin/grep/tests

2017-04-05 Thread Kyle Evans
On Wed, Apr 5, 2017 at 1:48 PM, Ngie Cooper (yaneurabeya) < yaneurab...@gmail.com> wrote: > > As a general rule of thumb, if it doesn’t pass, it needs to be > marked as an expected failure until it passes. Not really a big deal right > now (because ci.freebsd.org isn’t spamming current@ —

Re: svn commit: r316536 - in head: contrib/netbsd-tests/usr.bin/grep usr.bin/grep/tests

2017-04-05 Thread Ngie Cooper (yaneurabeya)
> On Apr 5, 2017, at 11:41, Ed Maste wrote: > > Author: emaste > Date: Wed Apr 5 18:41:44 2017 > New Revision: 316536 > URL: https://svnweb.freebsd.org/changeset/base/316536 > > Log: > bsdgrep: create additional tests for coverage on recent fixes > > Create additional tests to cover regress

svn commit: r316536 - in head: contrib/netbsd-tests/usr.bin/grep usr.bin/grep/tests

2017-04-05 Thread Ed Maste
Author: emaste Date: Wed Apr 5 18:41:44 2017 New Revision: 316536 URL: https://svnweb.freebsd.org/changeset/base/316536 Log: bsdgrep: create additional tests for coverage on recent fixes Create additional tests to cover regressions that were discovered by PRs linked to reviews D10098, D1