bug#28083: Two patches for grep on Windows, fixing directory recursion and leaking color

2017-11-26 Thread Stephan Lavavej
On Sun, Aug 13, 2017 at 3:53 PM, Stephan Lavavej wrote: > I have two patches that improve grep's support for Windows. The first > patch is a one-liner that fixes directory recursion, while the second > patch fixes "leaking color" when grep is printing colorized output and > is terminated premature

bug#28083: Two patches for grep on Windows, fixing directory recursion and leaking color

2017-11-26 Thread Jim Meyering
On Sun, Nov 26, 2017 at 1:10 AM, Stephan Lavavej wrote: > On Sun, Aug 13, 2017 at 3:53 PM, Stephan Lavavej wrote: >> I have two patches that improve grep's support for Windows. The first >> patch is a one-liner that fixes directory recursion, while the second >> patch fixes "leaking color" when g

bug#28083: Two patches for grep on Windows, fixing directory recursion and leaking color

2017-11-26 Thread Stephan Lavavej
On Sun, Nov 26, 2017 at 3:58 PM, Jim Meyering wrote: > [for reference, we're discussing these: > https://debbugs.gnu.org/db/28/28083.html] > Thank you for preparing those. You're welcome! > Would you please follow the contribution guidelines in grep's HACKING > file, https://git.savannah.gnu.org

bug#28083: Two patches for grep on Windows, fixing directory recursion and leaking color

2017-11-26 Thread Jim Meyering
On Sun, Nov 26, 2017 at 6:13 PM, Stephan Lavavej wrote: > On Sun, Nov 26, 2017 at 3:58 PM, Jim Meyering wrote: >> [for reference, we're discussing these: >> https://debbugs.gnu.org/db/28/28083.html] >> Thank you for preparing those. > > You're welcome! > >> Would you please follow the contributio

bug#28083: Two patches for grep on Windows, fixing directory recursion and leaking color

2017-11-26 Thread Stephan Lavavej
On Sun, Nov 26, 2017 at 3:58 PM, Jim Meyering wrote: > [for reference, we're discussing these: > https://debbugs.gnu.org/db/28/28083.html] On Sun, Nov 26, 2017 at 6:58 PM, Jim Meyering wrote: > I made only these changes to the commit log before pushing that: > - removed capitalization and traili