On Mon, Jun 8, 2015 at 11:43 AM, Luke Diamand <luke@xxxxxxxxxxx> wrote: > On 08/06/15 18:18, Junio C Hamano wrote: >> >> Lex Spoon <lex@xxxxxxxxxxxx> writes: >> >>> Precisely, Junio, that's what I had in mind. The patch with the two >>> lines deleted LGTM. >> >> >> Thanks, will do. > > > I don't think we're quite there yet unfortunately. > > The current version of git-p4 will let you do things like: > > $ git p4 clone //depot@1,2015/05/31 > > i.e. get all the revisions between revision 1 and the end of last month. > > Because my change tries to batch up the revisions, it fails when presented > with this. > > There aren't any test cases for this, but it's documented (briefly) in the > manual page. > > I think that although the current code looks really nice and clean, it's > going to have to pick up a bit more complexity to handle non-numerical > revisions. I don't think it's possible to do batching at the same time. > > It shouldn't be too hard though; I'll look at it later this week. [jch: adding git@ back] Thanks. -- 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