Hi again, Jonathan Nieder writes: > Ramkumar Ramachandra wrote: >> Jonathan Nieder writes: >>> The -s thing doesn't have much to do with this change. But is it a >>> bug or not? If it's not a bug, then this is not so much a glitch to >>> note as an important feature to ensure people don't sign off on a >>> conflict resolution without thinking about it. (I guess I think it's >>> a bug. It's hard to decide.) >> >> Bug, definitely. It happens because unlike "-x" where the >> cherry-picking machinery appends to the commit message, "-s" is >> handled at commit-time (when it spawns `git commit`). Ofcourse, if I >> were never to write the sequencing features, this would never been >> seen as a bug -- hence the term "glitch"; an implementation detail >> that doesn't suit our future plans (namely, this series) very well. > > Hmm, I thought I remembered this coming up previously and Junio > mentioning that it was intentional (maybe in the context of > CHERRY_HEAD?), but I haven't been able to find the relevant message. Hm, not automatically signing off on a commit resolution. I'm not sure I understand why. What does it mean to signoff? It proof of origin, more than anything else -- it means that you didn't pick up the code from some random place, right? Why would you want to signoff on commits you're picking from another branch? A topic branch that you want to merge into 'master' -- all the commits in the topic branch are yours (or you atleast know where the code came from), but there are probably commits from other developers in 'master'. So what happens when you resolve a conflict? 1. You modify some of your own code to fit into the code in 'master'. Auto-signoff is alright here. 2. You modify someone else's code to accommodate your code in 'master'. Here, your commit contains hunks which either modify or remove someone else's code, but how is this sort of thing different from a normal patch? Intent is different- I would've never made that change unless I tried accommodating this topic branch in 'master', but it has no effect on the origin of the code. Are you saying that auto-signoff is not alright here? Why? Wait. There's another explanation. You resolved the conflict and got everything into 'master' successfully, but some bugs crept into the program (that aren't present in the topic branch) because you resolved the conflict in a hurry. Still, what does all this have to do with certificate of origin? Thanks. -- Ram -- 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