Re: [RFC PATCH] Revamp git-cherry(1)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Nov 21, 2013 at 12:30:56PM +0100, Thomas Rast wrote:

> git-cherry(1)'s "description" section has never really managed to
> explain to me what the command does.  It contains too much explanation
> of the algorithm instead of simply saying what goals it achieves, and
> too much terminology that we otherwise do not use (fork-point instead
> of merge-base).
> 
> Try a much more concise approach: state what it finds out, why this is
> neat, and how the output is formatted, in a few short paragraphs.  In
> return, provide a longer example of how it fits into a format-patch/am
> based workflow.

FWIW, I find your concise explanation much friendlier.

> +Later, you can whether your changes have been applied by saying (still
> +on `topic`):

s/can/& see/ ?

> +------------
> +git fetch  # update your notion of origin/master
> +git cherry -v
> +------------
> ++
> +Note that this uses , and assumes that
> +`core.autosetupmerge` is enabled (the default).

I couldn't quite parse this. Is there a word missing before the comma,
or is it "uses and assumes that..."?

Given that it is the default, I wonder if it is worth mentioning at all.
Even I, who knows what autosetupmerge does, took a minute to figure out
why it is relevant here. I suspect it may just confuse most readers.

-Peff
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]