[GSoC Update] Week 3
- From
Pranit Bauva <pranit.bauva@gmail.com>
- Date
- May 22, 2016, 19:58 UTC
- Message-ID
- <1463947103-28464-1-git-send-email-pranit.bauva@gmail.com>
- In-Reply-To
- <1463334359-14033-1-git-send-email-pranit.bauva@gmail.com>
============================ 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:
* My patches for write_terms()[1] and bisect_log()/voc()[2] have collected some reviews from Eric Sunshine and Johannes Schindelin.
* As advised in the reviews I have mainly read about git_path() and other related functions from cache.h
* Also read up on the refs stuff like for_each_ref_in() which Junio mentioned in the previous update email.
* I have made 2 commits[3] which I am keeping on hold for now as they are followup for a commit made by Jeff King which I came across while reading about git_path() and related stuff.
* I have kind of finished writing bisect_clean_state() function. But I am still working on that as I suspect there might be something missing.
============================ NEXT STEPS ================================ Since I wasn't able to cover up many things this week (which I had initially planned), those things have now been carried forward along with a few other stuff. I plan to work more this week and the coming few ones to make up for the lost time.
Things which would be done in the coming week:
* Finish off (finally) bisect_clean_state() conversion.
* Convert the function bisect_head(). I plan to convert this function and add it as a subcommand to test the implementation but I will only send the function without the subcommand to the mailing list because its a too small function. Though the subcommand version will be put up on github for everyone to verify whether it is passing the test suite like I have done it for bisect_voc().
* Convert the function bisect_write(). I plan to convert this function and add it as a subcommand.
* Investigate why test no. 43 and 44 are failing in t6030 with `|| exit` in --write-terms.
[1]: http://thread.gmane.org/gmane.comp.version-control.git/294388 [2]: http://thread.gmane.org/gmane.comp.version-control.git/294571