From: vk Date: Tue, 26 Mar 2024 11:56:19 GMT Subject: Re: [GSoC] Microproject help Message-ID: In-Reply-To: On 2024-03-26 02:25, Karthik Nayak wrote: > vk writes: > > Hello! > >> Hello all, >> >> I'm Vk and I work as a software engineer. I am interested in >> contributing to git repository. Apologies for the late start as I only >> stumbled upon GSoC recently and git project interested me as I use it >> everyday at work and it would be meaningful for me to contribute to >> it. >> Even if I am not chosen, it will be great if I can start learning to >> contribute to git open source. >> >> For the microproject, I have looked into the lists for 2024 and it >> seems >> that all the projects have been taken except for `Replace a >> run_command*() call by direct calls to C functions`. However, it seems >> that the issue has been solved. The command I ran to search throughout >> the repo is `git grep 'run_command*('` and the search result returns >> run_command functions which I assume are essential. >> > > When I ran the same, I saw around 135 results. Are you saying that > they're all essential? > > For e.g. In `builtin/gc.c:maintenance_task_pack_refs`, shouldn't you be > able to replace the `run_command`? > My bad as the example uses `run_command_v_opt` and upon further inspection it seems that there might be similar situations like the example that can save an external process. However, for the maintenance_task_pack_refs, the command being run is `git pack-refs` which is a write process unlike the example which is a simple read only process of `git show-branch` to get the current commit? I will be searching the codebase for similar read only `run_command` and see if I can replace them. Thank you