All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ingo Molnar <mingo@elte.hu>
To: "H. Peter Anvin" <hpa@zytor.com>
Cc: Sam Ravnborg <sam@ravnborg.org>,
	Thomas Gleixner <tglx@linutronix.de>,
	Ingo Molnar <mingo@redhat.com>,
	LKML <linux-kernel@vger.kernel.org>,
	Jeff Dike <jdike@addtoit.com>,
	Linus Torvalds <torvalds@linux-foundation.org>,
	Andrew Morton <akpm@linux-foundation.org>
Subject: Re: [GIT PULL] x86: use arch/x86/include
Date: Wed, 30 Jul 2008 20:32:01 +0200	[thread overview]
Message-ID: <20080730183201.GA26389@elte.hu> (raw)
In-Reply-To: <4890B153.6050305@zytor.com>


* H. Peter Anvin <hpa@zytor.com> wrote:

> Ingo Molnar wrote:
>> * Sam Ravnborg <sam@ravnborg.org> wrote:
>>
>>> Hi x86 guys.
>>>
>>> It turned out to be easy to enable um to use arch/x86/include so here 
>>> is a git pull.
>>
>> hm, we have a _ton_ of changes to include files queued up already, so  
>> this is rather inconvenient.
>
> Git *should* be able to track those changes across a rename and even 
> with a rename on one branch and changes on another; in my experience 
> it works well for filename renames, but git doesn't understand 
> directory renames at all, so new files do have to be moved to their 
> new locations manually.

yes, it copes in some cases - but i've had rather bad experiences with 
it.

the reason i raised this is because i tried to pull Sam's renames, and 
they created 48 conflicts.

>> I missed the discussion on this, what's the point of renaming all these 
>> files?
>
> I know there has been talk about this on and off for a long time (to 
> get all the arch code into arch/).  I don't know if there are any 
> mechanical reasons for it, on top of that.

hm, seems rather pointless to me, i thought there might be some better 
reasons for it. Historically we've put all include files into 
include/asm-* - why upset the decade-long status quo now without strong 
technical reasons?

	Ingo

  reply	other threads:[~2008-07-30 18:32 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-07-30 12:49 [GIT PULL] x86: use arch/x86/include Sam Ravnborg
2008-07-30 18:04 ` Ingo Molnar
2008-07-30 18:22   ` H. Peter Anvin
2008-07-30 18:32     ` Ingo Molnar [this message]
2008-07-30 18:35     ` Linus Torvalds
2008-07-30 18:47   ` Sam Ravnborg
2008-07-30 19:19     ` Ingo Molnar
2008-07-30 19:27       ` H. Peter Anvin
2008-07-30 19:39       ` Sam Ravnborg
2008-07-31 23:15         ` Tony Luck
2008-07-31 23:17           ` H. Peter Anvin

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=20080730183201.GA26389@elte.hu \
    --to=mingo@elte.hu \
    --cc=akpm@linux-foundation.org \
    --cc=hpa@zytor.com \
    --cc=jdike@addtoit.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@redhat.com \
    --cc=sam@ravnborg.org \
    --cc=tglx@linutronix.de \
    --cc=torvalds@linux-foundation.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 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.