Re: [GSOC Update] Week 15

From: Pranit Bauva <pranit.bauva@gmail.com>
Subject: [GSOC Update] Week 15

=============== SUMMARY ================== My public git.git is available here[1]. I regularly keep pushing my work so anyone interested can track me there. Feel free to participate in the discussions going on PRs with my mentors. Your comments are valuable.

============== INTRODUCTION ============== The purpose of this project is to convert the git-bisect utility which partly exists in the form of shell scripts to C code so as to make it more portable. I plan to do this by converting each function to C and then calling it from git-bisect.sh so as to use the existing test suite to test the function which is converted.

Mentors: Christian Couder <chriscool@tuxfamily.org> Lars Schneider <larsxschneider@gmail.com>

================ Updates =================== Things which were done in this week:

 * I have converted bisect_start() and the bug has been eliminated too.

 * I have converted bisect_next() but still has some bugs. A notable
   problem is that the bisect.c code uses exit() statements and `trap` in
   the shell code for cleanup with `bisect_clean_state()`. So as
   suggested by Lars, I intend to use `bisect_clean_state()` along with
   die() as cleanup (or goto preferably).

================= NEXT STEPS ================= Things which would be done in the coming week:

 * Resend all patches rebased wtih v2.10-rc0.

 * bisect_next()

 * Following that I will convert bisect_auto_start()

 * Then bisect_replay().

=========== My Patches (GSoC project only) ===========

 * My current work is sent out to the mailing list here[2] which contains
   the whole conversion. Please don't merge the previous patches to next
   yet. Junio has requested me to rebase it on v2.10-rc0 so will resend
   it. Plus there are some change in the patch 04/13 which change the
   location of the function `bisect_clean_state()` from
   builtin/bisect--helper.c to bisect.c .

[1]: https://github.com/pranitbauva1997/git [2]: https://public-inbox.org/git/010201567675adc1-17e27495-6b36-40d1-836d-814da029fcc4-000000@eu-west-1.amazonses.com/

Regards, Pranit Bauva