From: "W. Trevor King" <wking@tremily.us>
To: Robert Dailey <rcdailey.lists@gmail.com>
Cc: Git <git@vger.kernel.org>
Subject: Re: Submodules with feature branches
Date: Thu, 5 Jun 2014 12:18:29 -0700 [thread overview]
Message-ID: <20140605191829.GA32192@odin.tremily.us> (raw)
In-Reply-To: <20140605190033.GV21803@odin.tremily.us>
[-- Attachment #1: Type: text/plain, Size: 1534 bytes --]
On Thu, Jun 05, 2014 at 12:00:33PM -0700, W. Trevor King wrote:
> On Thu, Jun 05, 2014 at 01:31:39PM -0500, Robert Dailey wrote:
> > Instead of just creating my branch and starting to make commits, I
> > now have to setup my submodule branch first. Also pull requests
> > won't show the changes to the third party libraries unless I do a
> > second pull request for the third party repo.
>
> That I agree with ;). However, if you're treating the third-party
> library as a separate repo, I think it makes sense that you need to
> be making branches and pull requests in the submodule independently
> from your branches and pull requests in the superproject.
To make this more concrete, I think you'll rarely have tight
one-to-one binding between third-party library changes and your
superproject. More likely, you'll have some high-level feature branch
in the superproject (“accept comments via email”) and an unrelated
number of prerequisite feature branches for your libraries (“add
support for MIME documents,” “parse RFC 2822 dates,” …). You only
have synchronized branches when you mess with the API tying components
together (updating the submodule API and updating the superproject to
use it). With good library design, that type of API migration should
happen more and more rarely as the library stabilizes.
Cheers,
Trevor
--
This email may be signed or encrypted with GnuPG (http://www.gnupg.org).
For more information, see http://en.wikipedia.org/wiki/Pretty_Good_Privacy
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]
prev parent reply other threads:[~2014-06-05 19:18 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-06-05 14:03 Submodules with feature branches Robert Dailey
2014-06-05 15:15 ` W. Trevor King
2014-06-05 15:57 ` Robert Dailey
2014-06-05 16:23 ` W. Trevor King
2014-06-05 18:31 ` Robert Dailey
2014-06-05 19:00 ` W. Trevor King
2014-06-05 19:18 ` W. Trevor King [this message]
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=20140605191829.GA32192@odin.tremily.us \
--to=wking@tremily.us \
--cc=git@vger.kernel.org \
--cc=rcdailey.lists@gmail.com \
/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).