/srv/irclogs.ubuntu.com/2017/07/07/#launchpad.txt

=== chihchun_afk is now known as chihchun
=== maclin1 is now known as maclin
=== chihchun is now known as chihchun_afk
naccso I'm 99% sure this diff is very wrong: https://code.launchpad.net/~nacc/ubuntu/+source/apache2/+git/apache2/+merge/327021 ... I can clone those two repos down locally and diff the two refs and I get: http://paste.ubuntu.com/25040631/ . Is this an artifact of the object sharing?17:37
nacccjwatson: --^?17:37
cjwatsonObject sharing is an unlikely culprit17:38
nacccjwatson: yeah it wasn't immediately clear why that would be, but it's the only thing i can think of :)17:38
cjwatsonExactly which diff syntax did you use?17:38
cjwatsonThe merge diff is actually done by "try a merge and then see what the diff in the merged index is"17:40
nacccjwatson: locally? `git diff --stat pkg/debian/sid..nacc/lp1702582` where pkg is the ~usd-import-team remote and nacc is the ~nacc remote for the apache2 src packageds17:41
naccah17:41
naccyeah we may be abusing the merge logical :)17:41
nacc*logic17:41
nacccjwatson: it's odd, though, the act of this merge should be a FF17:41
cjwatsonSince the whole point of it is to try to preview what would happen if you merged it17:41
nacccjwatson: as debian/sid is an immediate ancestor of lp1702582 in my repo17:41
cjwatsonIf you've managed to confuse git's merge algorithms somehow ...17:41
nacc(the same hash)17:41
nacccjwatson: let me see what happens locally17:41
nacccjwatson: it's a FF locally17:42
cjwatsonYeah, I don't see anything obviously wrong and it's EOW for me.  You could dig into lp:turnip (turnip/api/store.py:get_merge_diff) to see if you see anything?17:45
nacccjwatson: ack, thanks17:45
nacccjwatson: interesting, pygit2 is generating that output, from the index. Debugging why (merge_base on the same commits shows that debian/sid is the merge-base, which implies a FF)18:04
nacccjwatson: enjoy your w/e18:04
naccrbasak: --^ this might be something for your list, as you come up to speed on launchpad18:16
naccrbasak: i'm not sure why it worked in one case, but not in two18:16

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!