← Back to team overview

yade-dev team mailing list archive

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