Re: [PATCH] fast-import.c: always honor the filename case

From: Torsten Bögershausen <tboegi@web.de>
Date: Feb 11, 2014, 17:29 UTC
Message-ID: <52FA5DEF.30407@web.de>
In-Reply-To: <CAD_8n+ToUDbXrVuru7GV7toYKHXuQb8vL3B_-sfzQdXZFqzD2A@mail.gmail.com>

On 2014-02-10 15.24, Reuben Hawkins wrote:

Show 20 quoted lines

>
>
>
> On Sun, Feb 9, 2014 at 2:34 PM, Torsten Bögershausen <tboegi@web.de <mailto:tboegi@web.de>> wrote:
>
>     On 2014-02-06 12 <tel:2014-02-06%2012>.24, Reuben Hawkins wrote:
>     [snipped away minor interesting stuff]
>
>         Reading the answers from Peff and Junio, I am convinced that the fast-import should
>         not look at core.ignorecase at all.
>
>
>     Agreed, but my patch 0001-fast-import.c-ignorecase-iff-... is working very well (for me anyway), and the ignore-case option may be useful to the git-p4 importer (which I guess is something that should be determined for sure).
>
>     If you want, you can turn this into a real patch and send it to the list.
>     I think  Pete Wyckoff <pw@padd.com <mailto:pw@padd.com>> is one of the experts about p4.
>
>
> hehe. I thought it was a real patch....so I guess I'm not sure what you mean.  What do I need to do to make the patch more real (pardon my ignorance)?
> /Reuben

I have found a patch which was send as an attachment, and that's why it got the file name 0001-fast-import.c-ignorecase-iff.... here So yes, it was a real patch. Please re-send it inline, not as an attachment, as you did in "[PATCH] fast-import.c: always honor the filename case"

Show 9 quoted lines

>
>
>     And the same is for fast-export fixes you have made: If you have the time,
>     convert it into a patch and send it to the list.
>
>      
>
>
> Do you mean the fast-export fixes where I pushed all the delete operations to the front of the list?

Yes /Torsten