linux-next.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* linux-next: strange state of the kbuild tree
@ 2014-04-07 21:47 Stephen Rothwell
  2014-04-08  8:06 ` Michal Marek
  0 siblings, 1 reply; 3+ messages in thread
From: Stephen Rothwell @ 2014-04-07 21:47 UTC (permalink / raw)
  To: Michal Marek; +Cc: linux-next, linux-kernel

[-- Attachment #1: Type: text/plain, Size: 883 bytes --]

Hi Michal,

I noticed that you have two copies of two commits in your tree ("Kbuild,
lto: Set TMPDIR for LTO v4" and "Kbuild, lto: Add Link Time Optimization
support v3").  I understan why you recommitted them (to add your
Signed-off-by), but to leave the unsigned commits in there sort of
defeats the purpose.  This is probably a case where you should have
removed the original merge (or just these two commits) and then merged
the new version.

I am not sure what you intend to send to Linus, but this whole tree (in
one piece) should not be it.  From the state of the tree, it looks like
you send your separate topic branches (or a new merge of them) which
should be ok, but please after doing that, clean up your for-next branch
(it current has merge commits dating back to May, 2103 in it).
-- 
Cheers,
Stephen Rothwell                    sfr@canb.auug.org.au

[-- Attachment #2: Type: application/pgp-signature, Size: 836 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: linux-next: strange state of the kbuild tree
  2014-04-07 21:47 linux-next: strange state of the kbuild tree Stephen Rothwell
@ 2014-04-08  8:06 ` Michal Marek
  2014-04-08 10:42   ` Stephen Rothwell
  0 siblings, 1 reply; 3+ messages in thread
From: Michal Marek @ 2014-04-08  8:06 UTC (permalink / raw)
  To: Stephen Rothwell; +Cc: linux-next, linux-kernel

On 2014-04-07 23:47, Stephen Rothwell wrote:
> Hi Michal,
> 
> I noticed that you have two copies of two commits in your tree ("Kbuild,
> lto: Set TMPDIR for LTO v4" and "Kbuild, lto: Add Link Time Optimization
> support v3").  I understan why you recommitted them (to add your
> Signed-off-by), but to leave the unsigned commits in there sort of
> defeats the purpose.  This is probably a case where you should have
> removed the original merge (or just these two commits) and then merged
> the new version.

Sorry about that.


> I am not sure what you intend to send to Linus, but this whole tree (in
> one piece) should not be it.  From the state of the tree, it looks like
> you send your separate topic branches (or a new merge of them) which
> should be ok, but please after doing that, clean up your for-next branch
> (it current has merge commits dating back to May, 2103 in it).

Right, I send the individual topic branches to Linus. Will it be OK if I
reset the for-next branch to v3.15-rc1 after this merge window?

Thanks,
Michal

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: linux-next: strange state of the kbuild tree
  2014-04-08  8:06 ` Michal Marek
@ 2014-04-08 10:42   ` Stephen Rothwell
  0 siblings, 0 replies; 3+ messages in thread
From: Stephen Rothwell @ 2014-04-08 10:42 UTC (permalink / raw)
  To: Michal Marek; +Cc: linux-next, linux-kernel

[-- Attachment #1: Type: text/plain, Size: 321 bytes --]

Hi Michal,

On Tue, 08 Apr 2014 10:06:09 +0200 Michal Marek <mmarek@suse.cz> wrote:
>
> Right, I send the individual topic branches to Linus. Will it be OK if I
> reset the for-next branch to v3.15-rc1 after this merge window?

Yep, fine.

-- 
Cheers,
Stephen Rothwell                    sfr@canb.auug.org.au

[-- Attachment #2: Type: application/pgp-signature, Size: 836 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2014-04-08 10:42 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-04-07 21:47 linux-next: strange state of the kbuild tree Stephen Rothwell
2014-04-08  8:06 ` Michal Marek
2014-04-08 10:42   ` Stephen Rothwell

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).