From: greened@obbligato.org (David A. Greene)
To: Ramkumar Ramachandra <artagnon@gmail.com>
Cc: David Greene <dag@cray.com>,
git@vger.kernel.org, Junio C Hamano <gitster@pobox.com>
Subject: Re: git-subtree
Date: Thu, 05 Jan 2012 10:33:20 -0600 [thread overview]
Message-ID: <87sjjuyur3.fsf@smith.obbligato.org> (raw)
In-Reply-To: <CALkWK0k+AwCsizZFwbKKxuz0B4xLoyC4hAy3WRD=sLCq-HvvCw@mail.gmail.com> (Ramkumar Ramachandra's message of "Thu, 5 Jan 2012 21:02:16 +0530")
Ramkumar Ramachandra <artagnon@gmail.com> writes:
> Hi again,
>
> [+CC: Junio Hamano, our maintainer]
>
> David A. Greene wrote:
>> I've read that document. The issue is that I didn't develop the code,
>> Avery did.
>
> Not an issue as long as you have Avery's signoff.
As in a signed-off-by log entry on the commit? I did a commit -s to add
my own signed-off-by tag and added a "From:" line in accordance with the
SubmittingPatches document:
"If you are forwarding a patch from somebody else, optionally, at the
beginning of the e-mail message just before the commit message starts,
you can put a "From: " line to name that person."
I have not used signoffs before in my day-to-day git flow. How do I go
about getting one from Avery and incorporating it into the history in an
autheticated way? I'm assuming you don't want me to forge his sign-off.
:)
>> It's a lot of time to learn a
>> completely new codebase. I was hoping to submit something soon and then
>> learn the codebase gradually during maintenance/further development.
>
> We certainly don't want badly reviewed code that nobody understands
> floating around in the codebase-
Certainly, I'm not trying to avoid review, just trying to figure out the
most efficient mechanics.
> so, I'd suggest sending out whatever you think is appropriate for the
> first round of reviews, and see how things shape up from there.
Fair enough. I think I will take Jeff's suggested route and see where
that goes.
>> How have completely new tools be introduced into the git mainline in the
>> past?
>
> Yes. For an example of something I was involved with but didn't
> author, see vcs-svn/.
Ok, I'll look into that. Thanks for the pointer.
-Dave
next prev parent reply other threads:[~2012-01-05 22:24 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-01-04 15:53 git-subtree David Greene
2012-01-05 11:28 ` git-subtree Ramkumar Ramachandra
2012-01-05 15:03 ` git-subtree David A. Greene
2012-01-05 15:32 ` git-subtree Ramkumar Ramachandra
2012-01-05 16:33 ` David A. Greene [this message]
2012-01-06 1:53 ` git-subtree David A. Greene
2012-01-05 22:18 ` git-subtree David A. Greene
2012-01-05 15:47 ` git-subtree Jeff King
2012-01-05 16:26 ` git-subtree David A. Greene
2012-01-29 22:07 ` git-subtree David A. Greene
2012-01-30 16:56 ` git-subtree David A. Greene
2012-01-05 22:16 ` git-subtree David A. Greene
2012-01-05 15:53 ` git-subtree Junio C Hamano
2012-01-05 16:48 ` git-subtree David A. Greene
2012-01-05 22:19 ` git-subtree David A. Greene
-- strict thread matches above, loose matches on Subject: below --
2021-11-14 10:11 git-subtree tqfx su
2021-11-15 8:12 ` git-subtree Fabian Stelzer
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=87sjjuyur3.fsf@smith.obbligato.org \
--to=greened@obbligato.org \
--cc=artagnon@gmail.com \
--cc=dag@cray.com \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.