Re: [PATCH] Document patch syntax.

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

 



On 2007-05-12 14:07:04 +0200, Yann Dirson wrote:

> +Patches in the stack are identified with a short name, which must be
> +unique in the stack.

s/a short name/short names/, maybe.

> +Patches in the current stack are just refered to by their name.  Some
> +commands allow to specify a patch in another stack of the repository;

s/allow to/allow you to/. Or "allows one to", but I prefer the second
person here.

> +this is done by suffixing the patch name with a '@' sign followed by the
> +branch name (eg. 'thispatch@otherbranch').

s/a '@'/an '@'/; "'@'" begins with a vowel sound. :-)

> +A number of position in the stack related to the patch are also

s/position/positions/.

> +accessible through '//' suffixes.  For example, 'patch//top'' is
> +equivalent to 'patch', and 'patch//bottom' refers to the commit below
> +'patch' (ie. the patch below, or the stack base if this is the

s/ie./i.e./.

> +bottom-most patch).  Similarly ''//top.old'' and ''//bottom.old''

You use double quotes here, and single quotes above. What's the
distinction?

> +refer to the previous version of the patch (before the last
> +stglink:push[] or stglink:refresh[] operation).  When refering to the

s/refering/referring/.

Otherwise,

Acked-by: Karl Hasselström <kha@xxxxxxxxxxx>

Yes, I'm too picky.

-- 
Karl Hasselström, kha@xxxxxxxxxxx
      www.treskal.com/kalle
-
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]

  Powered by Linux