gitster@xxxxxxxxx wrote on Wed, 03 Jul 2013 13:33 -0700: > Jonathan Nieder <jrnieder@xxxxxxxxx> writes: > > > Michael Schubert wrote: > > > >> --- a/Documentation/git-p4.txt > >> +++ b/Documentation/git-p4.txt > >> @@ -180,7 +180,7 @@ subsequent 'sync' operations. > >> Import changes into given branch. If the branch starts with > >> 'refs/', it will be used as is. Otherwise if it does not start > >> with 'p4/', that prefix is added. The branch is assumed to > >> - name a remote tracking, but this can be modified using > >> + name a remote-tracking, but this can be modified using > >> '--import-local', or by giving a full ref name. The default > >> branch is 'master'. > > > > This is confusing both before and after the patch. What is "a remote > > tracking"? > > > > Perhaps: > > > > --branch <ref>:: > > Import changes into <ref> instead of refs/remotes/p4/master. > > If <ref> starts with refs/, it is used as is. Otherwise, if > > it does not start with p4/, that prefix is added. > > + > > By default a <ref> not starting with refs/ is treated as the > > name of a remote-tracking branch (under refs/remotes/). This > > behavior can be modified using the --import-local option. > > + > > The default <ref> is "master". > > > > The rest of the patch looks good. > > Myy reading did hiccup at the same "remote-tracking" used as if it > were a noun, and your rewritten version reads much better. Yes, very clear and complete rewrite; thanks. The final paragraph is perhaps duplicative of the first sentence, but adds clarity, so I'm happy as it stands. Acked-by: Pete Wyckoff <pw@xxxxxxxx> -- Pete -- 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