On 16/08/2012 17:06, Thilina Buddika Abeyrathna wrote:
>
>
> On Thu, Aug 16, 2012 at 2:57 AM, Thilina Buddika Abeyrathna
> <thilinaabeyrathna@gmail.com <mailto:thilinaabeyrathna@gmail.com>> wrote:
>
>
>
> On Thu, Aug 16, 2012 at 2:55 AM, Marc Delisle <marc@infomarc.info
> <mailto:marc@infomarc.info>> wrote:> <mailto:rouslan@placella.com>>
>
> Le 2012-08-15 16:24, Thilina Buddika Abeyrathna a écrit :
> > Hi Marc,
> > I'm looking [1] bug in the table operation page,
> > I 'git checkout' to very old commit and check the same bug in
> there, But
> > still this bug was there,
> > I found that, first bad commit was
> 4d2fcd910bc7e07114fcc76f55705c1a6e8c685e
> > (This is not a my commit)
> > I can't figure out where is this error happen.
> >
> > [1] -
> >
> http://sourceforge.net/tracker/?func=detail&aid=3555235&group_id=23067&atid=377408
>
> Thilina,
> please use "git bisect" [2] to find the problematic commits. I used
>
> git bisect start HEAD QA_3_5
>
> and found this:
>
> a16e723048940f87b4be4b4097d16c4376826f2c is the first bad commit
> commit a16e723048940f87b4be4b4097d16c4376826f2c
> Author: Rouslan Placella <rouslan@placella.com
> Date: Mon Jun 11 17:58:35 2012 +0100Problem #1 is already fixed in my branch [0], so I guess that you can
>
> Dropped PMA_ajaxResponse()
>
> [2] http://wiki.phpmyadmin.net/pma/Git#Finding_problematic_commit
>
>
> Thank you Marc.
>
> Hi Rouslan,
> Can you look at this error?
ignore it.
Problem #2 has nothing to do with my code. Also keep in mind that the
issue comes up when changing from innodb to myisam, but not the other
way around. Here's the results of a bisect:
83401195710de736dc6217f885313b5dad9071ae is the first bad commit
commit 83401195710de736dc6217f885313b5dad9071ae
Author: Thilina Buddika <thilinaabeyrathna@gmail.com>
Date: Wed Aug 1 13:52:49 2012 +0530
further refactoring for PMA_getTableOptionFieldset()
:040000 040000 f7dba249f1ec47de37b26a2df7fc4562dd717b80
7b7945eceffca79186916f9ed7e4e30f0d4cb0d7 M libraries
Bye,
Rouslan
[0]: https://github.com/roccivic/phpmyadmin/tree/frames-removal