mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Linus Torvalds <torvalds@linux-foundation.org>
To: Steven Rostedt <rostedt@goodmis.org>
Cc: Stefan Richter <stefanr@s5r6.in-berlin.de>,
	Theodore Tso <tytso@mit.edu>, Ingo Molnar <mingo@elte.hu>,
	LKML <linux-kernel@vger.kernel.org>,
	Nicolas Pitre <nico@fluxnic.net>,
	"Luck, Tony" <tony.luck@intel.com>,
	Stephen Rothwell <sfr@canb.auug.org.au>,
	"Luis R. Rodriguez" <mcgrof@gmail.com>,
	Jeff Garzik <jeff@garzik.org>,
	Robert Richter <robert.richter@amd.com>,
	Dmitry Torokhov <dmitry.torokhov@gmail.com>,
	Jean Delvare <khali@linux-fr.org>,
	Sam Ravnborg <sam@ravnborg.org>
Subject: Re: [RFC] to rebase or not to rebase on linux-next
Date: Tue, 27 Oct 2009 12:42:45 -0700 (PDT)	[thread overview]
Message-ID: <alpine.LFD.2.01.0910271232200.31845@localhost.localdomain> (raw)
In-Reply-To: <1256671624.26028.419.camel@gandalf.stny.rr.com>



On Tue, 27 Oct 2009, Steven Rostedt wrote:
>
> On Tue, 2009-10-27 at 12:06 -0700, Linus Torvalds wrote:
> >  Don't lie about 
> > getting an ack that you didn't get before you made that patch public.
> 
> But how do you get your ack without making it public?

There's a difference between exposing your git tree to the public, and 
showing your patches to others.

A public git tree is _not_ the place to ask for comments for patches. If 
you haven't gotten Ack's, you have two choices:

 - that commit should not show up in a tree that is marked for -next, 
   because it's still waiting for feedback and people to test it. It may 
   be in a _private_ git tree of course, but it's not "ready" yet.

 - you are going to commit it regardless of acks or not, and you don't 
   care, and you shouldn't lie about it later.

Those are the two obvious choices. Adding ack's later, after it has 
already been exported and tested in -next is kind of pointless, isn't it? 
It's basically lying about how the patch came to be.

If you want Ack's from people, then send that patch around by email.

NOTE! If you know the people you want acks from are git users, then the 
email can certainly be something like "look at so-and-so branch of my git 
tree <here>". You can certainly use private git branches as a way to talk 
to other developers about code you're not fully happy with yet, with the 
clear understanding that you want acks and comments on it.

Put another way: there's a big conceptual difference between "public git 
tree" and "private git branch that may well be available to others and 
is meant for development". In git itself, Junio has the 'pu' branch that 
is clearly marked as being rebased etc.

				Linus

  parent reply	other threads:[~2009-10-27 19:43 UTC|newest]

Thread overview: 36+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <4ADD0793.3000707@kernel.org>
     [not found] ` <20091020020449.GF24370@core.coreip.homeip.net>
     [not found]   ` <20091020034829.GA12833@elte.hu>
     [not found]     ` <20091020140750.GH11972@erda.amd.com>
     [not found]       ` <alpine.LFD.2.00.0910210654480.5125@eeepc.linux-foundation.org>
     [not found]         ` <20091022122042.e535d43c.sfr@canb.auug.org.au>
     [not found]           ` <20091023112732.GB5886@elte.hu>
     [not found]             ` <4AE19A74.1090709@garzik.org>
     [not found]               ` <20091023123555.GA25366@elte.hu>
     [not found]                 ` <57C9024A16AD2D4C97DC78E552063EA3E33D0174@orsmsx505.amr.corp.intel.com>
     [not found]                   ` <20091023134115.GD27097@elte.hu>
     [not found]                     ` <alpine.LFD.2.00.0910231424440.21460@xanadu.home>
     [not found]                       ` <20091023191631.GA1879@elte.hu>
2009-10-23 19:35                         ` [RFC] to rebase or not to rebase on liunx-next Steven Rostedt
2009-10-23 20:37                           ` [RFC] to rebase or not to rebase on linux-next Ingo Molnar
2009-10-23 20:54                           ` Ingo Molnar
2009-10-23 21:59                             ` Sam Ravnborg
2009-10-26 23:26                               ` Steven Rostedt
2009-10-26 23:30                                 ` David Miller
2009-10-26 23:51                                   ` Steven Rostedt
2009-10-27  0:15                                     ` David Miller
2009-10-27  0:30                                       ` Steven Rostedt
2009-10-27  1:34                                         ` David Miller
2009-10-27  3:02                                           ` Steven Rostedt
2009-10-27  4:56                                             ` Theodore Tso
2009-10-27  5:18                                             ` David Miller
2009-10-27  8:13                                               ` Ingo Molnar
2009-10-27  9:48                                       ` Catalin Marinas
2009-10-27  7:59                                 ` Ingo Molnar
2009-10-27 15:39                                   ` Steven Rostedt
2009-10-27 17:18                                     ` Nicolas Pitre
2009-10-28 15:15                                       ` Steven Rostedt
2009-10-28  6:40                                     ` Ingo Molnar
2009-10-24  8:03                             ` Theodore Tso
2009-10-24 12:20                               ` Stefan Richter
2009-10-24 19:43                                 ` Sam Ravnborg
2009-10-27 19:06                                 ` Linus Torvalds
2009-10-27 19:27                                   ` Steven Rostedt
2009-10-27 19:39                                     ` Valdis.Kletnieks
2009-10-27 19:43                                       ` Steven Rostedt
2009-10-27 19:42                                     ` Linus Torvalds [this message]
2009-10-26  4:53                               ` Luck, Tony
2009-10-26  5:21                                 ` Theodore Tso
2009-10-26 18:01                                   ` Stefan Richter
2009-10-27 18:07                                     ` Theodore Tso
2009-10-27 18:54                                       ` Stefan Richter
2009-10-28  7:26                                       ` Ingo Molnar
2009-10-24 12:51                           ` Stefan Richter
     [not found]                       ` <4AE21839.3000000@s5r6.in-berlin.de>
2009-10-23 21:03                         ` [kernel.org users] Please remember to run 'git gc' on your repositories Steven Rostedt

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=alpine.LFD.2.01.0910271232200.31845@localhost.localdomain \
    --to=torvalds@linux-foundation.org \
    --cc=dmitry.torokhov@gmail.com \
    --cc=jeff@garzik.org \
    --cc=khali@linux-fr.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mcgrof@gmail.com \
    --cc=mingo@elte.hu \
    --cc=nico@fluxnic.net \
    --cc=robert.richter@amd.com \
    --cc=rostedt@goodmis.org \
    --cc=sam@ravnborg.org \
    --cc=sfr@canb.auug.org.au \
    --cc=stefanr@s5r6.in-berlin.de \
    --cc=tony.luck@intel.com \
    --cc=tytso@mit.edu \
    /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

Powered by JetHome