linux-next.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Jean Delvare <khali@linux-fr.org>
To: Stephen Rothwell <sfr@canb.auug.org.au>
Cc: linux-next@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: linux-next: rebase of the jdelvare-hwmon quilt series
Date: Thu, 5 Sep 2013 09:25:36 +0200	[thread overview]
Message-ID: <20130905092536.44dbe86a@endymion.delvare> (raw)
In-Reply-To: <20130905101903.0198dbbfdf9e9172fb9b55b2@canb.auug.org.au>

Hi Stephen,

On Thu, 5 Sep 2013 10:19:03 +1000, Stephen Rothwell wrote:
> Why did you bother rebasing the jdelvare-hwmon quilt series?  None of the
> patches changed (not that that even matters).

I'm not "rebasing". Rebasing is a git thing, and I'm using quilt, not
git, to feed linux-next. Until the patches hit Linus' tree, I am not
maintaining a tree, I am maintaining a patch set on top of a moving
tree. I only use git to send my pull request to Linus to make things
easier for him.

What you see is just how quilt works. That's nothing to worry about.
I've been working that way for over 6 years. I am curious why you
started yelling at me about this 3 months ago when it has proven to
work just fine for 6 years. You asked me to tag what my quilt series
was based on, and I did. You asked me to get my patches in linux-next
ASAP so that they get more testing, and I do.

Last time, you mentioned a problem with testing results being lost when
I "rebase" my quilt series". I don't quite think it's true for hwmon
patches as the subsystem is pretty much self-contained (the things I am
working on at least) plus I do built-test and to some extent run-test
the patches continuously, so I am confident my approach isn't breaking
bisectability.

But I have heard you nevertheless and already slightly changed my
process. I will no longer be "rebasing" during the merge window, i.e.
the first pull request I send during the merge window, which contains
the patches that have been waiting or have been added since the
previous merge window, will now be based on the just-released .0
kernel, rather than the current pre-rc1 state of Linus' tree at the
time I issue the pull request. Hopefully this should address your "test
results are lost" concern.

I don't think there's much more I can offer. I have to work with a
relatively recent tree so I have to "rebase" from times to times
anyway. At least once per development cycle, right? And I don't think
it would be very reasonable to stay on -rc1 all the time. That's why I
follow upstream development between -rc1 and final, and will only stop
doing so between final and -rc1. Staying with .0 wouldn't be reasonable
either, there's a reason why we maintain stable kernels trees. I don't
think you want developers to base their work on the previous stable
kernel, right?

You see, I don't want to make your life harder or anything. It's just
that your request doesn't make sense to me. "Don't rebase" is not
something I can do much with, as I do have to follow upstream
development and rebase at some point in time anyway. If you propose a
complete workflow which is better than what I do today, I'll be happy
to consider switching to it. But just telling me to skip one mandatory
step in my current workflow is simply not helping, sorry.

-- 
Jean Delvare

  reply	other threads:[~2013-09-05  7:25 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-09-05  0:19 linux-next: rebase of the jdelvare-hwmon quilt series Stephen Rothwell
2013-09-05  7:25 ` Jean Delvare [this message]
  -- strict thread matches above, loose matches on Subject: below --
2013-07-11  0:27 Stephen Rothwell
2013-07-11  5:57 ` Jean Delvare
2013-07-11  6:53   ` Stephen Rothwell
2013-07-11  7:39     ` Jean Delvare
2013-07-11 23:17       ` Paul Gortmaker
2013-07-12  0:27         ` Stephen Rothwell
2013-07-12  0:36           ` Stephen Rothwell

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=20130905092536.44dbe86a@endymion.delvare \
    --to=khali@linux-fr.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-next@vger.kernel.org \
    --cc=sfr@canb.auug.org.au \
    /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).