lkml.org 
[lkml]   [2008]   [Jul]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [GIT PULL] MMC updates for 2.6.26
On Thu, Jul 17, 2008 at 1:41 AM, Pierre Ossman <drzeus-list@drzeus.cx> wrote:
> On Wed, 16 Jul 2008 15:16:26 -0700 (PDT)
> Linus Torvalds <torvalds@linux-foundation.org> wrote:
>
>>
>>
>> On Tue, 15 Jul 2008, Pierre Ossman wrote:
>> >
>> > 45 files changed, 6054 insertions(+), 1320 deletions(-)
>>
>> If you had done the "--summary", you'd also have seen these bogus:
>>
>> mode change 100644 => 100755 drivers/mmc/core/sdio_io.c
>> mode change 100644 => 100755 include/linux/mmc/sdio_func.h
>>
>> entries.
>>
>> I have no idea why you think those things should be executable, but it
>> happened in commit 6d37333163025b46afbcad434ec9a5f2e88e7254 ("mmc: fix
>> sdio_io sparse errors").
>>
>
> Ouch. It was in the patch submitted to me, but I overlooked the mode
> changes there. I wasn't fully aware git could express such things in
> the patches. :)
>
>> Anyway, there's a reason why I ask people to use
>>
>> --diffstat --summary -M
>>
>> when generating the report. This is just part of it.
>>
>> I fixed it up in the merge, so it's ok in my tree, but I thought I'd point
>> it out.
>>
>
> Thanks. I have changed my scripts and hopefully I'll notice such
> silliness in the future.
>
> Rgds
> --
> -- Pierre Ossman

I take the blame on me. I've copied that file between OSes so it
changed executable. It slipped from my attention
Will also change my scripts as well.

Tomas


\
 
 \ /
  Last update: 2008-07-17 08:41    [W:0.063 / U:0.212 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site