>On 30/06/17 18:28, Stefan Beller wrote:
The patch makes a lot of sense - thanks for the fast reply.
A question: does the header correspond to the patch ?
< [PATCH] status: suppress additional warning output in plumbing modes
> [PATCH] status: suppress CRLF warnings in porcelain modes
(And may be the comment in the code:)
< / * suppress all additional output in porcelain mode */
> / * suppress CRLF conversion warnings in porcelain mode */
When status is called with '--porcelain' (as implied by '-z'), we promise
to output only messages as described in the man page.
Suppress CRLF warnings.
Signed-off-by: Stefan Beller <sbeller@xxxxxxxxxx>
---
Maybe something like this?
builtin/commit.c | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/builtin/commit.c b/builtin/commit.c
index 00a01f07c3..3705d5ec6f 100644
--- a/builtin/commit.c
+++ b/builtin/commit.c
@@ -1126,6 +1126,11 @@ static void finalize_deferred_config(struct wt_status *s)
die(_("--long and -z are incompatible"));
}
+ /* suppress all additional output in porcelain mode */
+ if (status_format == STATUS_FORMAT_PORCELAIN ||
+ status_format == STATUS_FORMAT_PORCELAIN_V2)
+ safe_crlf = SAFE_CRLF_FALSE;
+
if (use_deferred_config && status_format == STATUS_FORMAT_UNSPECIFIED)
status_format = status_deferred_config.status_format;
if (status_format == STATUS_FORMAT_UNSPECIFIED)
