Re: [GSoC] Microproject help

[vk <g@vkabc.me>]

On 2024-03-26 02:25, Karthik Nayak wrote:
> vk <g@vkabc.me> 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