Re: [PATCH] Improve and fix git-check-attr(1)

From: Junio C Hamano <gitster@pobox.com>
Date: Oct 14, 2008, 23:58 UTC
Message-ID: <7vabd6686n.fsf@gitster.siamese.dyndns.org>
In-Reply-To: <20081013175705.GA9538@diku.dk>

Jonas Fonseca <fonseca@diku.dk> writes:

Show 13 quoted lines

> Always use 'verse' for multi-line synopsis sections. Add output and
> example sections to document what output can be expected.
>
> Signed-off-by: Jonas Fonseca <fonseca@diku.dk>
>
> ---
>  Documentation/git-check-attr.txt |   63 +++++++++++++++++++++++++++++++++++++-
>  1 files changed, 62 insertions(+), 1 deletions(-)
>
>  At least the first chunk should be applied. The last is a quick attempt
>  at documenting the expected output formally and with examples.
> ...
> tg: (340fcf4..) jf/man-git-check-attr (depends on: next)

Should this really have to depend on next? I have a feeling that a large part of this (sans --stdin/-z) is a 'maint' material.

Could you split this into two patches, one for 'maint' to describe the output format, and another for 'next' (or 'dp/checkattr') to update the SYNOPSIS part?