# Bug report 6 messages from 2017-08-30 to 2017-09-02. Participants: Aleksandar Pavic, Kevin Daudt, Dov Grobgeld, Stephan Beyer, Jeff King. Thread: https://gitlist.dev/t/46688 ## Aleksandar Pavic, 2017-08-30 21:25 Subject: Bug report Message-ID: URL: https://gitlist.dev/e/db0580dc-3ce0-47ba-b82a-b1e50b9d0a3a%40nsinfo.co.rs ``` I have a file app/Controller/CustomerCardVerificationController.php And when I take a look at changes log, I get this (no matter which tool I use): 2017-07-31 19:41 dule o membership renew payment email 2017-06-07 08:59 Dusan Tatic o cc refund clean 2017-04-15 00:16 Miodrag Dragić o refound admin payment 2017-03-20 12:02 Dusan Tatic o CardVerification card connect 2017-03-16 15:59 Aleksandar Pavic o paypal 2017-03-10 13:34 Aleksandar Pavic o Production branch 2017-03-10 13:01 Aleksandar Pavic I Migrating dev However if I manually browse thru revisions and open revision from 03/27/2017 07:05 PM I can see the change in that file which is unlisted above, at revision ff9f4946e109bd234d438e4db1d319b1f6cb6580 And I'm at master branch all the time... We wouldn't have noticed that if it weren't one important feature... ``` ## Kevin Daudt, 2017-08-31 06:36 Subject: Re: Bug report Message-ID: <20170831063614.GI3839@alpha.vpn.ikke.info> URL: https://gitlist.dev/e/20170831063614.GI3839%40alpha.vpn.ikke.info In-Reply-To: ``` On Wed, Aug 30, 2017 at 11:25:00PM +0200, Aleksandar Pavic wrote: > I have a file > > app/Controller/CustomerCardVerificationController.php > > And when I take a look at changes log, I get this (no matter which tool I > use): > > 2017-07-31 19:41 dule o membership renew payment email > 2017-06-07 08:59 Dusan Tatic o cc refund clean > 2017-04-15 00:16 Miodrag Dragić o refound admin payment > 2017-03-20 12:02 Dusan Tatic o CardVerification card connect > 2017-03-16 15:59 Aleksandar Pavic o paypal > 2017-03-10 13:34 Aleksandar Pavic o Production branch > 2017-03-10 13:01 Aleksandar Pavic I Migrating dev > > However if I manually browse thru revisions and open revision from > 03/27/2017 07:05 PM > > I can see the change in that file which is unlisted above, at revision > ff9f4946e109bd234d438e4db1d319b1f6cb6580 > > And I'm at master branch all the time... > > We wouldn't have noticed that if it weren't one important feature... > What does git branch --contains ff9f4946e109bd234d438e4db1d319b1f6cb6580 return? Where did you find this commit? ``` ## Dov Grobgeld, 2017-08-31 14:19 Subject: Re: Bug report Message-ID: URL: https://gitlist.dev/e/CA%2B%2BfsGHFwoLvwO121aDEgi1dZbwwfogdWbxxRx%2Bci7T1RDCAnw%40mail.gmail.com In-Reply-To: <20170831063614.GI3839@alpha.vpn.ikke.info> ``` The following answer that I got back in 2015 from Jeff King might be relevant to your problem: https://marc.info/?l=git&m=144178948506788&w=2 Regards, Dov On Thu, Aug 31, 2017 at 9:36 AM, Kevin Daudt wrote: > On Wed, Aug 30, 2017 at 11:25:00PM +0200, Aleksandar Pavic wrote: >> I have a file >> >> app/Controller/CustomerCardVerificationController.php >> >> And when I take a look at changes log, I get this (no matter which tool I >> use): >> >> 2017-07-31 19:41 dule o membership renew payment email >> 2017-06-07 08:59 Dusan Tatic o cc refund clean >> 2017-04-15 00:16 Miodrag Dragić o refound admin payment >> 2017-03-20 12:02 Dusan Tatic o CardVerification card connect >> 2017-03-16 15:59 Aleksandar Pavic o paypal >> 2017-03-10 13:34 Aleksandar Pavic o Production branch >> 2017-03-10 13:01 Aleksandar Pavic I Migrating dev >> >> However if I manually browse thru revisions and open revision from >> 03/27/2017 07:05 PM >> >> I can see the change in that file which is unlisted above, at revision >> ff9f4946e109bd234d438e4db1d319b1f6cb6580 >> >> And I'm at master branch all the time... >> >> We wouldn't have noticed that if it weren't one important feature... >> > > What does git branch --contains ff9f4946e109bd234d438e4db1d319b1f6cb6580 > return? > > Where did you find this commit? ``` ## Aleksandar Pavic, 2017-08-31 14:55 Subject: Re: Bug report Message-ID: URL: https://gitlist.dev/e/e334fd40-0bf2-21d8-e647-ef1195911e04%40nsinfo.co.rs In-Reply-To: ``` Hm, thanks, but the link was not helpful, there are no merge commits to master branch. Those changes should have not been undone, that's how we caught it, because 1 line peace of code introduced new feature (+-10% tolerance on credit card verification amount). So most likeley they were undone, but our tree is pretty simple, we have master and production branches. On 31.08.2017. 16:19, Dov Grobgeld wrote: > The following answer that I got back in 2015 from Jeff King might be > relevant to your problem: > > https://marc.info/?l=git&m=144178948506788&w=2 > > Regards, > Dov > > On Thu, Aug 31, 2017 at 9:36 AM, Kevin Daudt wrote: >> On Wed, Aug 30, 2017 at 11:25:00PM +0200, Aleksandar Pavic wrote: >>> I have a file >>> >>> app/Controller/CustomerCardVerificationController.php >>> >>> And when I take a look at changes log, I get this (no matter which tool I >>> use): >>> >>> 2017-07-31 19:41 dule o membership renew payment email >>> 2017-06-07 08:59 Dusan Tatic o cc refund clean >>> 2017-04-15 00:16 Miodrag Dragić o refound admin payment >>> 2017-03-20 12:02 Dusan Tatic o CardVerification card connect >>> 2017-03-16 15:59 Aleksandar Pavic o paypal >>> 2017-03-10 13:34 Aleksandar Pavic o Production branch >>> 2017-03-10 13:01 Aleksandar Pavic I Migrating dev >>> >>> However if I manually browse thru revisions and open revision from >>> 03/27/2017 07:05 PM >>> >>> I can see the change in that file which is unlisted above, at revision >>> ff9f4946e109bd234d438e4db1d319b1f6cb6580 >>> >>> And I'm at master branch all the time... >>> >>> We wouldn't have noticed that if it weren't one important feature... >>> >> What does git branch --contains ff9f4946e109bd234d438e4db1d319b1f6cb6580 >> return? >> >> Where did you find this commit? ``` ## Stephan Beyer, 2017-08-31 16:23 Subject: Re: Bug report Message-ID: URL: https://gitlist.dev/e/be66e3d8-3673-fe5e-c65d-2dc9a7202315%40gmx.net In-Reply-To: <20170831063614.GI3839@alpha.vpn.ikke.info> ``` On 08/31/2017 08:36 AM, Kevin Daudt wrote: > On Wed, Aug 30, 2017 at 11:25:00PM +0200, Aleksandar Pavic wrote: >> I have a file >> >> app/Controller/CustomerCardVerificationController.php >> >> And when I take a look at changes log, I get this (no matter which tool I >> use): >> >> 2017-07-31 19:41 dule o membership renew payment email >> 2017-06-07 08:59 Dusan Tatic o cc refund clean >> 2017-04-15 00:16 Miodrag Dragić o refound admin payment >> 2017-03-20 12:02 Dusan Tatic o CardVerification card connect >> 2017-03-16 15:59 Aleksandar Pavic o paypal >> 2017-03-10 13:34 Aleksandar Pavic o Production branch >> 2017-03-10 13:01 Aleksandar Pavic I Migrating dev >> >> However if I manually browse thru revisions and open revision from >> 03/27/2017 07:05 PM >> >> I can see the change in that file which is unlisted above, at revision >> ff9f4946e109bd234d438e4db1d319b1f6cb6580 I am not sure if I fully understand but I guess your commit ff9f4946e1 has been commited at detached HEAD. You could do git log --oneline --graph master ff9f4946e1 to see where the "missing" commit lies. And probably a git cherry-pick ff9f4946e1 might pick your missing feature. Stephan ``` ## Jeff King, 2017-09-02 08:49 Subject: Re: Bug report Message-ID: <20170902084944.2mw64l5wod6vj3fh@sigill.intra.peff.net> URL: https://gitlist.dev/e/20170902084944.2mw64l5wod6vj3fh%40sigill.intra.peff.net In-Reply-To: ``` On Wed, Aug 30, 2017 at 11:25:00PM +0200, Aleksandar Pavic wrote: > I have a file > > app/Controller/CustomerCardVerificationController.php > > And when I take a look at changes log, I get this (no matter which tool I > use): > > 2017-07-31 19:41 dule o membership renew payment email > 2017-06-07 08:59 Dusan Tatic o cc refund clean > 2017-04-15 00:16 Miodrag Dragić o refound admin payment > 2017-03-20 12:02 Dusan Tatic o CardVerification card connect > 2017-03-16 15:59 Aleksandar Pavic o paypal > 2017-03-10 13:34 Aleksandar Pavic o Production branch > 2017-03-10 13:01 Aleksandar Pavic I Migrating dev > > However if I manually browse thru revisions and open revision from > 03/27/2017 07:05 PM > > I can see the change in that file which is unlisted above, at revision > ff9f4946e109bd234d438e4db1d319b1f6cb6580 How are you invoking the log? Are you doing: git log app/Controller/CustomerCardVerificationController.php or similar? If that is the case, then history simplification may be causing the results you see. And even you don't _see_ any merges in the output, that is because they were simplified away. And the commit you are looking for may have been on a side branch that was simplified away. If you do: git log --full-history app/... does the commit you are interested in show up? If so, then it was removed due to history simplification. And if you are surprised that a side branch was simplified away, that is most likely because there is a mis-merge in your history (some merge which threw away the changes on a side branch). Try: git log --graph --oneline --name-status --full-history app/... to see the whole shape of history, including which commits touched the file. You can read more about it in the "History Simplification" section of "git help log". -Peff ```