git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* Re: git-p4: Fix sync errors due to new server version
@ 2010-02-01 19:38 David Aguilar
  2010-02-01 19:57 ` Nicolas Pitre
  0 siblings, 1 reply; 3+ messages in thread
From: David Aguilar @ 2010-02-01 19:38 UTC (permalink / raw)
  To: gitster; +Cc: simon, git, pal_engstad


Hi Junio

It looks like we forgot about this patch:

http://article.gmane.org/gmane.comp.version-control.git/137723

The patch looks good and Pal added his sign-off
(I believe that and not inlining the patch were the original
 setbacks).

Simon, are there any reasons to not apply it?

Thanks,

-- 
		David

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: git-p4: Fix sync errors due to new server version
  2010-02-01 19:38 git-p4: Fix sync errors due to new server version David Aguilar
@ 2010-02-01 19:57 ` Nicolas Pitre
  2010-02-01 20:47   ` Junio C Hamano
  0 siblings, 1 reply; 3+ messages in thread
From: Nicolas Pitre @ 2010-02-01 19:57 UTC (permalink / raw)
  To: David Aguilar; +Cc: gitster, simon, git, pal_engstad

On Mon, 1 Feb 2010, David Aguilar wrote:

> 
> Hi Junio
> 
> It looks like we forgot about this patch:
> 
> http://article.gmane.org/gmane.comp.version-control.git/137723
> 
> The patch looks good and Pal added his sign-off
> (I believe that and not inlining the patch were the original
>  setbacks).
> 
> Simon, are there any reasons to not apply it?

FWIW the patch looks fine to me.  Not that I'm affected by the new 
server version, but the patch can't break anything.


Nicolas

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: git-p4: Fix sync errors due to new server version
  2010-02-01 19:57 ` Nicolas Pitre
@ 2010-02-01 20:47   ` Junio C Hamano
  0 siblings, 0 replies; 3+ messages in thread
From: Junio C Hamano @ 2010-02-01 20:47 UTC (permalink / raw)
  To: Nicolas Pitre; +Cc: David Aguilar, simon, git, pal_engstad

Thanks; will queue it to 'master'.

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2010-02-01 20:48 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-02-01 19:38 git-p4: Fix sync errors due to new server version David Aguilar
2010-02-01 19:57 ` Nicolas Pitre
2010-02-01 20:47   ` Junio C Hamano

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).