On Thu, Apr 25, 2013 at 4:41 PM, Felipe Contreras <felipe.contreras@xxxxxxxxx> wrote: > On Thu, Apr 25, 2013 at 4:11 PM, Junio C Hamano <gitster@xxxxxxxxx> wrote: >> Thomas Rast <trast@xxxxxxxxxxx> writes: >> >>> Felipe Contreras <felipe.contreras@xxxxxxxxx> writes: >>> >>>> Not just randomly synchronize the revisions with no checks at all. This >>>> is the way bazaar's UI does it. >>>> >>>> Also, add a non-ff check. >>>> >>>> Signed-off-by: Felipe Contreras <felipe.contreras@xxxxxxxxx> >>>> --- >>>> >>>> This patch should probably go to maint, as the results of pushing the way we >>>> currently push are not really understood. Perhaps it's similar to a 'git push >>>> --force', or perhaps it can potentially screw the repository. >>>> >>>> It's better to be safe and just do what bazaar does. >>> >>> Other than "this patch should probably go to maint", this should be in >>> the commit message. >> >> Hmph, should it? >> >> I do not quite understand what "... are not really understood. >> Perhaps..." wants to say. Understood by whom? By the author of the >> patch? By the author of the original code? >> >> The log would end up saying "Doing the same as bazaar should be the >> right thing to do(TM), but don't ask me why. I do not know what I >> am doing, or why checking is better than not checking, but it seems >> to work". That _could_ be the truth, but it won't help people who >> are reading the code later, will it? > > In addition to that, it's doubtful there's anybody who would actually > know. Bazaar is basically a dead and unmaintained project now, and it > has been for quite some time. It seems the people that had any idea > are long gone. > > My gut feeling is that we should do it the way the Bazaar UI does it, > I don't have any evidence that there's anything wrong with the current > code, which Bazaar seems to but have, but for different purposes which > are hard to explain. I would rather avoid surprises. Er,which Bazaar seems to use, but for different purposes... -- Felipe Contreras -- 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