yade-dev team mailing list archive
-
yade-dev team
-
Mailing list archive
-
Message #14801
Re: git bisect is useful. The TwoPhaseFlowEngine.cpp warnings in --check
Really good to know, thanks.
Le dim. 5 mai 2019 à 17:56, Janek Kozicki <janek_listy@xxxxx> a écrit :
> Hi,
>
> I just realized how useful git bisect is. Bruno asked me this:
>
> > Do you know when this started to appear by any chance:
>
> > running: check_TwoPhaseFlowEngine_PressureInjection.py
> > WARN
> /home/3S-LAB/bchareyre/yade/trunk/trunk-fresh/pkg/pfv/TwoPhaseFlowEngine.cpp:2663
> clusterOutvadePore: index mismatch wrt. cell ids
>
> And thanks to ccache I could do a quick bisect search on master
> branch. I checked out from about a month ago. Did compile, ran tests.
> Then checked out 2 weeks ago. And I found that this problem was
> introduced by: e4bcc0c0
>
> I recommend reading `man git-bisect` :-)
>
> Also, this wouldn't have worked if some of the commits were committed
> without making sure that yade is compiling :)
>
> Janek
>
> _______________________________________________
> Mailing list: https://launchpad.net/~yade-dev
> Post to : yade-dev@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~yade-dev
> More help : https://help.launchpad.net/ListHelp
>
References