Homestyx hydra
Diffusion hydra d5163d014345

Write patterns to "git grep" on stdin instead of passing them with "-e"

Description

Write patterns to "git grep" on stdin instead of passing them with "-e"

Summary:
Fixes T12807. Some shells may apparently mangle/strip UTF8 characters? Just dodge this whole problem by sending the pattern over stdin rather than actually figuring out the particulars.

Related tasks, like T7339 and T5554, discuss finding broader fixes for this class of issue, and this definitely isn't exactly a fully legitimate fix, but in many cases (as here) we can reasonably just avoid the problem rather than actually fixing it, at least for a long time.

Test Plan: Searched for emoji and non-emoji locally, but this worked fine (on OSX) for me before the patch too.

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T12807

Differential Revision: https://secure.phabricator.com/D18105

Details

Provenance
epriestleyAuthored on Jun 8 2017, 8:53 AM
sirocylPushed on Oct 16 2024, 5:49 AM
Parents
R1:e26cd3ffe0f9: Give "x committed <commit>" feed stories an explicitly higher action strength…
Branches
Unknown
Tags
Unknown

Event Timeline