<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Thu, May 16, 2013 at 4:31 PM, Isaac Bennetch <span dir="ltr"><<a href="mailto:bennetch@gmail.com" target="_blank">bennetch@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote"><div class="im">On Thu, May 16, 2013 at 4:08 PM, Marc Delisle <span dir="ltr"><<a href="mailto:marc@infomarc.info" target="_blank">marc@infomarc.info</a>></span> wrote:<br>


<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Isaac Bennetch a écrit :<br>
<div><div>> Hi, I just merged the pull request for AES_ENCRYPT [1], which required a<br>
> little bit of manual work to complete the merge. I thought everything went<br>
> well, except the commit history [2] looks strange to me. Would someone mind<br>
> double-checking that I didn't inadvertently do something wrong? I pretty<br>
> much followed the GitHub instructions, but this is the first time I've done<br>
> a merge where I committed back to the project repository rather than my<br>
> private one, so I'm a bit extra cautious.<br>
><br>
> Thanks<br>
><br>
> 1 - <a href="https://github.com/phpmyadmin/phpmyadmin/pull/290" target="_blank">https://github.com/phpmyadmin/phpmyadmin/pull/290</a><br>
> 2 -<br>
> <a href="https://github.com/phpmyadmin/phpmyadmin/commit/8fbad56812f0b5b0730b07961c7db329bdef9dae" target="_blank">https://github.com/phpmyadmin/phpmyadmin/commit/8fbad56812f0b5b0730b07961c7db329bdef9dae</a><br>
<br>
</div></div>Hi Isaac,<br>
indeed it looks strange. I would say that you merged master into his<br>
feature branch before merging back to master.<br></blockquote><div><br></div></div><div>Thanks. In the interest of not making this worse, I think the right thing to do is git revert 16febdb. That will take me back to the last commit before my mistake, "There should be message variable as well". Then I have to commit and push that revert. Then I can redo the ayusun merge correctly. I just want to make positively sure that I do the revert correctly.<br>

</div></div></div></div></blockquote><div><br></div><div>After a bit more testing, I think the even better response is to git reset --hard 16feb and push that change. One part of the git documentation suggests one, another part suggests the other. I'm not clear on what the differences are.<br>

<br>The problem is that when I do a git diff and git status to make sure of what I'm about to push, it looks right...but then again, it also did when I messed things up. <br></div></div><br></div><div class="gmail_extra">

Any opinions?<br></div></div>