From: Junio C Hamano <gitster@pobox.com>
To: Miklos Vajna <vmiklos@frugalware.org>
Cc: Sean <seanlkml@sympatico.ca>, git@vger.kernel.org
Subject: Re: [PATCH] Add [HOWTO] using merge subtree.
Date: Tue, 08 Jan 2008 17:35:33 -0800 [thread overview]
Message-ID: <7vwsqjpyqi.fsf@gitster.siamese.dyndns.org> (raw)
In-Reply-To: <1199838182-15178-1-git-send-email-vmiklos@frugalware.org> (Miklos Vajna's message of "Wed, 9 Jan 2008 01:23:02 +0100")
Miklos Vajna <vmiklos@frugalware.org> writes:
NAK. It may be well intentioned, but it lacks too much context
to be usable as a generic teaching material.
> +Abstract: In this article, Sean demonstrates how one can use the subtree merge
> + strategy.
> +Message-ID: <BAYC1-PASMTP12374B54BA370A1E1C6E78AE4E0@CEZ.ICE>
> +
> +How to use the subtree merge strategy
> +=====================================
Here, before diving to the actual command sequence, you need to
tell the reader what the objective of the whole exercise is.
What's /path/to/B and how its contents relate to the contents of
the current repository? How was that other repository prepared?
What's the ultimate goal, IOW, what overlayed tree layout are
you trying to create? Why do you want to have that overlayed
layout?
When you heard it on the mailing list you may have had enough
background material from the discussion thread. Your readers
won't have that.
> +----
> +$ git remote add -f B /path/to/B <1>
> +$ git merge -s ours --no-commit B/master <2>
> +$ git read-tree --prefix=B/ -u B/master <3>
> +$ git commit -m "Merge commit 'B/master'" <4>
> +----
> +<1> creates and fetches the remote.
> +<2> initiates a merge, but stops before committing it.
> +<3> reads B/master into the subdirectory "sub".
> +<4> all that remains is committing the completed merge.
> +
> +You only need this procedure for the first commit, after which `git merge -s
> +subtree B/master` is all you need.
Complaints for an overly long line aside, after that
description, having another code example block to illustrate the
exact command line would be more consistent with the above
display. Alternatively, make it <5> and annotate that entry
with "You only need the above four steps once, after that you
can keep doing 'git pull -s subtree B master'".
next parent reply other threads:[~2008-01-09 1:36 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <1199838182-15178-1-git-send-email-vmiklos@frugalware.org>
2008-01-09 1:35 ` Junio C Hamano [this message]
2008-01-09 3:46 ` [PATCH] Add [HOWTO] using merge subtree Junio C Hamano
2008-01-09 6:28 ` Junio C Hamano
[not found] <1199882097-10420-1-git-send-email-vmiklos@frugalware.org>
2008-01-09 22:33 ` Junio C Hamano
2008-01-15 0:35 ` Miklos Vajna
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=7vwsqjpyqi.fsf@gitster.siamese.dyndns.org \
--to=gitster@pobox.com \
--cc=git@vger.kernel.org \
--cc=seanlkml@sympatico.ca \
--cc=vmiklos@frugalware.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).