kernelnewbies.kernelnewbies.org archive mirror
 help / color / mirror / Atom feed
From: greg@kroah.com (Greg KH)
To: kernelnewbies@lists.kernelnewbies.org
Subject: Need help: Generating patch using git
Date: Wed, 1 Feb 2012 14:05:54 -0800	[thread overview]
Message-ID: <20120201220554.GC893@kroah.com> (raw)
In-Reply-To: <CAC_BnSV3pyqpssh4y2xWoDbk0Dxi3KQPhNYdS7ZHKa0yp3RJUQ@mail.gmail.com>

On Thu, Feb 02, 2012 at 02:47:49AM +0530, Srivatsa Bhat wrote:
> Hi,
> 
> On Wed, Feb 1, 2012 at 10:51 AM, amit mehta <gmate.amit@gmail.com> wrote:
> 
> 
>     > Also the kernel tree you are using seems to be Linus's mainline, is
>     > that what you wanted or did you want to be making the patch against a
>     > linux-next kernel?
> 
>     My current goal is to send some patches to kernel janitor group though I'm
>     not
>     sure if this group is still active or not.
>     you mean to say that this is not the tree which i should be synced to? If
>     not
>     then can you please send me the link to the relevant git repository ?
> 
> 
> 
> Please note that linux-next is just a tree used for integration-testing. I
> strongly suggest
> that you don't base your patches on linux-next. Basing it on current mainline
> is generally a good idea. But if you are doing some significant development,
> you should target the individual trees that the subsystem maintainers maintain.
> 
> To put it in simple terms, base your patch on current mainline and send it to
> the appropriate people (use get_maintainer.pl in the scripts directory to find
> whom to send it to). Then if the maintainer specifically asks you to rebase
> your
> patch on some particular tree that he maintains, then do it. Then you know what
> to do with patches related to that subsystem from next time onwards :-)

As a subsystem maintainer, I strongly disagree with this.

Do your work against linux-next, as that contains the different
subsystems already.  You don't want to do something only to find out you
need to totally redo it, or just throw it away as someone else has
already done it (which is quite common for janitorial and other "simple"
tasks).

So please, either work against linux-next, or the subsystem-specific
tree, linux-next is usually easier, the odds of cross-subsystem merges
causing problems with your change, for the subsystem maintainer, are
very low, much lower than the fact that major changes might have already
happened.

greg k-h

  reply	other threads:[~2012-02-01 22:05 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-01-30 17:21 Need help: Generating patch using git amit mehta
2012-01-30 17:27 ` Subramaniam Appadodharana
2012-01-30 17:32 ` Srivatsa Bhat
2012-01-30 18:33   ` amit mehta
2012-01-31 21:39     ` Julie Sullivan
2012-02-01  5:21       ` amit mehta
2012-02-01 21:01         ` Julie Sullivan
2012-03-05 20:48           ` Amit Mehta
2012-03-07 21:58             ` Julie Sullivan
2012-03-08  7:46               ` amit mehta
2012-02-01 21:17         ` Srivatsa Bhat
2012-02-01 22:05           ` Greg KH [this message]
2012-02-01 22:12             ` Srivatsa Bhat

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=20120201220554.GC893@kroah.com \
    --to=greg@kroah.com \
    --cc=kernelnewbies@lists.kernelnewbies.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).