diff options
author | Elliott Hughes <enh@google.com> | 2020-11-06 10:25:37 -0800 |
---|---|---|
committer | Rob Landley <rob@landley.net> | 2020-11-07 01:54:01 -0600 |
commit | e1a5111cbbfe18707fb26a65ed7af654f30a132c (patch) | |
tree | 7b2d9a38baa0230fbee47914e164f1f578e23677 /toys/other | |
parent | 90c4e155dd0acf79f4f725c9dcd985304824c156 (diff) | |
download | toybox-e1a5111cbbfe18707fb26a65ed7af654f30a132c.tar.gz |
xregcomp: add the specific regex we couldn't parse to the error message.
This makes it more likely that you can actually debug something like
"sed: bad regex: empty (sub)expression" or
"sed: bad regex: parentheses not balanced" from a build failure log,
where you don't necessarily know where the failure came from.
This also seems like it might be useful generally, although GNU grep
doesn't include this detail in its error messages, and busybox doesn't
even seem to notice that regcomp() failed?
(Realistically if the Android build team wants to move forward with "one
true regex implementation", we're going to have to add some GNU-isms to
the Android regex implementation. But we'd need to find them first! Note
that the two examples given above are real examples from failed buildbot
builds, but they occur early in the respective builds so there are
likely many more to look at after these. Interestingly, the first of the
two appears to be the more general case of something disallowed by POSIX
that xregcomp() already has a workaround for.)
Diffstat (limited to 'toys/other')
0 files changed, 0 insertions, 0 deletions