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
next prev parent 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.