Re: [patch 00/16] Portability Patches for git-1.7.1 (v4)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Michael,

Thanks for putting up a public repo of those patches; much
appreciated.

On Sat, May 01, 2010 at 04:25:12PM -0700, Junio C Hamano wrote:
> Michael J Gruber <git@xxxxxxxxxxxxxxxxxxxx> writes:
> > Gary V. Vaughan venit, vidit, dixit 27.04.2010 15:57:
> > > Here are the portability patches we needed at TWW to enable git-1.7.1
> > > to compile and run on all of the wide range of Unix machines we
> > > support. [[...]]
> >
> > OK, unsurprsingly, tests still pass on Linux (Fedora 12 x86_64).
> > If anyone wants to try, the series can be found at
> >
> > git://repo.or.cz/git/mjg.git
> >
> > in branch
> >
> > gvv/platform-compatibility
> >
> > applied cleanly on current master.
> 
> Thanks.  

Hi Junio,

> Like everybody else, I like the s/diff/test_cmp/ one in general.  On
> platforms without "diff -u", test_cmp may want to use "diff -c" instead
> for readability, but that can be fixed/enhanced independently.
> 
> I agree with your general remark and also noticed that the titles are way
> suboptimal.

I think this is my first patch series submission to the git list, but
in any case I just used used the patch file-names as is my habit for
locally maintained patchsets.

Thanks for suggesting better title, which I've already adopted into my
local tree.  If upstream integration stalls, or if it will help smooth
integration, I'll make the merging and reordering adjustments you
request and resubmit the whole queue.

Cheers,
-- 
Gary V. Vaughan (gary@xxxxxxxxxxxxxxxxxx)
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]