Re: [PATCH] Correction to git-p4 "exclude" change

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

 



(Resending as plain text).

I could be wrong about this, but my correction above doesn't seem to
be in 'next'. Does that mean (reading your last "what's cooking") that
the broken version is going to go out to 'master' soon?

Thanks,
Luke

On 5 February 2015 at 08:19, Luke Diamand <luke@xxxxxxxxxxx> wrote:
> I could be wrong about this, but my correction above doesn't seem to be in
> 'next'. Does that mean (reading your last "what's cooking") that the broken
> version is going to go out to 'master' soon?
>
> Thanks,
> Luke
>
>
> On 28 January 2015 at 20:49, Junio C Hamano <gitster@xxxxxxxxx> wrote:
>>
>> Luke Diamand <luke@xxxxxxxxxxx> writes:
>>
>> > My previous change for adding support for "exclude" to git-p4 "sync"
>> > was incorrect, missing out a comma, which stopped git-p4 from working.
>> > This change fixes that.
>> >
>> > I've also noticed that t9814-git-p4-rename.sh has stopped working; I'm
>> > going to follow up with a fix for that once I've worked out what's
>> > wrong with it. There's a small shell syntax problem (missing "esac")
>> > but after fixing that it still fails, so I'm not sure what's happening
>> > yet. It was discussed a while back.
>> >
>> > Luke Diamand (1):
>> >   git-p4: correct "exclude" change
>> >
>> >  git-p4.py | 2 +-
>> >  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> Thanks.
>>
>> Will keep out of 'master' for now.  It may not be a bad idea to
>> squash this fix (and any futher ones if needed) into a single patch
>> when we rewind 'next' after 2.3 final is tagged.
>>
>
--
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]