tree aab355630214867c937cb4bcaa3185782dd9f2c5
parent 99394d7bec91cb700a693b901c2807722db117ea
author Oliver Newman <olivernewman@google.com> 1691191178 +0000
committer CQ Bot <fuchsia-internal-scoped@luci-project-accounts.iam.gserviceaccount.com> 1691191178 +0000

[scm] Handle warnings in git diff output

The `git diff` output can sometimes include non-fatal warning messages
at the end. For example, if the diff contains at least
`diff.renameLimit` files, the following messages will show up
(newline-separated, even with the `-z` flag) at the end of the stdout:

  warning: exhaustive rename detection was skipped due to too many files.
  warning: you may want to set your diff.renameLimit variable to at least 2458 and retry the command.

We should ignore these messages as long as `git diff` produces a retcode
of zero, rather than erroring out.

Change-Id: I59fa60a36fa8bc76d7f517332588667bea171f8b
Reviewed-on: https://fuchsia-review.googlesource.com/c/shac-project/shac/+/892540
Fuchsia-Auto-Submit: Oliver Newman <olivernewman@google.com>
Commit-Queue: Auto-Submit <auto-submit@fuchsia-infra.iam.gserviceaccount.com>
Reviewed-by: Marc-Antoine Ruel <maruel@google.com>